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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/io_thread.h

Issue 2462983003: Move data use measurement to DataUseNetworkDelegate (Closed)
Patch Set: Addressed ryansturm comments 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
« chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc ('K') | « chrome/browser/extensions/api/web_request/web_request_api_unittest.cc ('k') | chrome/browser/io_thread.cc » ('j') | components/data_use_measurement/content/content_url_request_classifier.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/io_thread.h
diff --git a/chrome/browser/io_thread.h b/chrome/browser/io_thread.h
index b5a2a9def93e80f18abf90160f4cb8201049ad0d..134a87f6ff7e233edb376d5ba3bcfb1475894b48 100644
--- a/chrome/browser/io_thread.h
+++ b/chrome/browser/io_thread.h
@@ -24,6 +24,7 @@
#include "build/build_config.h"
#include "chrome/browser/net/chrome_network_delegate.h"
#include "chrome/common/features.h"
+#include "components/metrics/data_use_tracker.h"
Not at Google. Contact bengr 2016/11/11 00:09:48 Not required?
Raj 2016/11/11 19:10:39 Still needed. |metrics_data_use_forwarder_| is use
#include "components/prefs/pref_member.h"
#include "components/ssl_config/ssl_config_service_manager.h"
#include "content/public/browser/browser_thread.h"
@@ -399,7 +400,7 @@ class IOThread : public content::BrowserThreadDelegate {
const base::TimeTicks creation_time_;
// Callback for updating data use prefs which needs to be initialized on UI
- // thread and passed to |ChromeNetworkDelegate|.
+ // thread and passed to |DataUseNetworkDelegate|.
metrics::UpdateUsagePrefCallbackType metrics_data_use_forwarder_;
base::WeakPtrFactory<IOThread> weak_factory_;
« chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc ('K') | « chrome/browser/extensions/api/web_request/web_request_api_unittest.cc ('k') | chrome/browser/io_thread.cc » ('j') | components/data_use_measurement/content/content_url_request_classifier.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698