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

Unified Diff: chrome/browser/net/chrome_network_delegate.cc

Issue 1818613002: Implement UMA log throttling for cellular connections (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix build for gn, fix unittests, add comments Created 4 years, 9 months 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: chrome/browser/net/chrome_network_delegate.cc
diff --git a/chrome/browser/net/chrome_network_delegate.cc b/chrome/browser/net/chrome_network_delegate.cc
index ee6dff8e6cfcd91d612f1c1152431c0d502033e5..bb0c71f57e5e8d8f65c585d2b520f99f824f9112 100644
--- a/chrome/browser/net/chrome_network_delegate.cc
+++ b/chrome/browser/net/chrome_network_delegate.cc
@@ -286,7 +286,8 @@ void RecordCacheStateStats(const net::URLRequest* request) {
ChromeNetworkDelegate::ChromeNetworkDelegate(
extensions::EventRouterForwarder* event_router,
- BooleanPrefMember* enable_referrers)
+ BooleanPrefMember* enable_referrers,
+ metrics::UpdateUsagePrefCallbackType metrics_data_use_forwarder)
: profile_(NULL),
enable_referrers_(enable_referrers),
enable_do_not_track_(NULL),
@@ -295,13 +296,16 @@ ChromeNetworkDelegate::ChromeNetworkDelegate(
url_blacklist_manager_(NULL),
domain_reliability_monitor_(NULL),
experimental_web_platform_features_enabled_(
- base::CommandLine::ForCurrentProcess()
- ->HasSwitch(switches::kEnableExperimentalWebPlatformFeatures)),
+ base::CommandLine::ForCurrentProcess()->HasSwitch(
+ switches::kEnableExperimentalWebPlatformFeatures)),
data_use_aggregator_(nullptr),
is_data_usage_off_the_record_(true) {
DCHECK(enable_referrers);
extensions_delegate_.reset(
ChromeExtensionsNetworkDelegate::Create(event_router));
+
+ // Pass callback for updating data use prefs to |DataUseMeasurement| object.
+ data_use_measurement_.SetMetricsDataUseForwarder(metrics_data_use_forwarder);
Alexei Svitkine (slow) 2016/03/31 04:32:25 Can this be a ctor param for that object instead o
gayane -on leave until 09-2017 2016/03/31 17:54:57 Done.
}
ChromeNetworkDelegate::~ChromeNetworkDelegate() {}

Powered by Google App Engine
This is Rietveld 408576698