Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(613)

Unified Diff: components/data_use_measurement/core/data_use_network_delegate.h

Issue 2462983003: Move data use measurement to DataUseNetworkDelegate (Closed)
Patch Set: Rebased and fixed nits Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/data_use_measurement/core/data_use_network_delegate.h
diff --git a/components/data_use_measurement/core/data_use_network_delegate.h b/components/data_use_measurement/core/data_use_network_delegate.h
index 298a69079876bdc2b2700e26112d1c50c355e545..1aea4e6a5dadec4404af7f95f4a91ba4cf24d806 100644
--- a/components/data_use_measurement/core/data_use_network_delegate.h
+++ b/components/data_use_measurement/core/data_use_network_delegate.h
@@ -10,6 +10,8 @@
#include <memory>
#include "base/macros.h"
+#include "components/data_use_measurement/core/data_use_measurement.h"
+#include "components/metrics/data_use_tracker.h"
#include "net/base/completion_callback.h"
#include "net/base/layered_network_delegate.h"
@@ -29,7 +31,9 @@ class DataUseNetworkDelegate : public net::LayeredNetworkDelegate {
public:
DataUseNetworkDelegate(
std::unique_ptr<net::NetworkDelegate> nested_network_delegate,
- DataUseAscriber* ascriber);
+ DataUseAscriber* ascriber,
+ std::unique_ptr<URLRequestClassifier> url_request_classifier,
+ const metrics::UpdateUsagePrefCallbackType& metrics_data_use_forwarder);
~DataUseNetworkDelegate() override;
@@ -47,10 +51,15 @@ class DataUseNetworkDelegate : public net::LayeredNetworkDelegate {
void OnNetworkBytesSentInternal(net::URLRequest* request,
int64_t bytes_sent) override;
+ void OnCompletedInternal(net::URLRequest* request, bool started) override;
+
void OnURLRequestDestroyedInternal(net::URLRequest* request) override;
private:
DataUseAscriber* ascriber_;
+
+ // Component to report data use UMA.
+ data_use_measurement::DataUseMeasurement data_use_measurement_;
};
} // namespace data_use_measurement

Powered by Google App Engine
This is Rietveld 408576698