| commit | 91a655d210d4672bc6d84cb5b24866256a8d305e | [log] [tgz] |
|---|---|---|
| author | Julia Tuttle <[email protected]> | Fri Jan 26 18:03:03 2018 |
| committer | Commit Bot <[email protected]> | Fri Jan 26 18:03:03 2018 |
| tree | 3ac84f4613fb42dc2a13b47302d70e2ee449bd70 | |
| parent | fc3f45c02e43760981314c95ffeaca5ffaebebc4 [diff] [blame] |
Network Error Logging: Move base::Feature check to NetworkContext. Cronet doesn't support controlling base::Features, so move the base::Feature to //services/network (in a tiny component, because things break otherwise), and move the feature check itself to NetworkContext. (A future change will add an experimental option to Cronet to enable Reporting and NEL.) This is a fixed re-land of https://chromium-review.googlesource.com/c/chromium/src/+/858176. Bug: 799616 Change-Id: I79c8a011c75d6b79a92f9ffe85bf5d53d509dcbc Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Reviewed-on: https://chromium-review.googlesource.com/879441 Commit-Queue: Julia Tuttle <[email protected]> Reviewed-by: Jochen Eisinger <[email protected]> Reviewed-by: Matt Menke <[email protected]> Reviewed-by: Bernhard Bauer <[email protected]> Cr-Commit-Position: refs/heads/master@{#532006}
diff --git a/net/reporting/reporting_service.cc b/net/reporting/reporting_service.cc index 59976e23..46fbba8e 100644 --- a/net/reporting/reporting_service.cc +++ b/net/reporting/reporting_service.cc
@@ -62,6 +62,10 @@ return context_->uploader()->RequestIsUpload(request); } + const ReportingPolicy& GetPolicy() const override { + return context_->policy(); + } + private: std::unique_ptr<ReportingContext> context_;