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

Unified Diff: chrome/browser/io_thread.cc

Issue 1393073002: Add external data use observer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@datause_accounting_scliitle_cl_do_not_edit_2
Patch Set: Addressed comments Created 5 years, 2 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/io_thread.cc
diff --git a/chrome/browser/io_thread.cc b/chrome/browser/io_thread.cc
index 165bf37aaafac32b370109297fbcd70c6021dfa4..dc7b4e11a60b3a27722b19f3a48fc95a4982bb15 100644
--- a/chrome/browser/io_thread.cc
+++ b/chrome/browser/io_thread.cc
@@ -40,7 +40,6 @@
#include "chrome/common/pref_names.h"
#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_prefs.h"
#include "components/data_reduction_proxy/core/common/data_reduction_proxy_params.h"
-#include "components/data_usage/core/data_use_aggregator.h"
#include "components/net_log/chrome_net_log.h"
#include "components/policy/core/common/policy_service.h"
#include "components/proxy_config/pref_proxy_config_tracker.h"
@@ -108,6 +107,7 @@
#if defined(OS_ANDROID)
#include "base/android/build_info.h"
+#include "chrome/browser/android/datausage/external_data_use_observer_android.h"
#include "chrome/browser/android/net/external_estimate_provider_android.h"
#endif
@@ -568,7 +568,7 @@ void IOThread::Init() {
extension_event_router_forwarder_;
#endif
- globals_->data_use_aggregator.reset(new data_usage::DataUseAggregator());
+ data_use_aggregator_.reset(new data_usage::DataUseAggregator());
// TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466432
// is fixed.
@@ -580,8 +580,13 @@ void IOThread::Init() {
&system_enable_referrers_));
// By default, data usage is considered off the record.
chrome_network_delegate->set_data_use_aggregator(
- globals_->data_use_aggregator.get(),
- true /* is_data_usage_off_the_record */);
+ data_use_aggregator(), true /* is_data_usage_off_the_record */);
+
+#if defined(OS_ANDROID)
+ external_data_use_observer_.reset(
+ new chrome::android::ExternalDataUseObserverAndroid(
+ data_use_aggregator()));
+#endif
// TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466432
// is fixed.
@@ -1057,6 +1062,10 @@ base::TimeTicks IOThread::creation_time() const {
return creation_time_;
}
+data_usage::DataUseAggregator* IOThread::data_use_aggregator() const {
+ return data_use_aggregator_.get();
+}
+
net::SSLConfigService* IOThread::GetSSLConfigService() {
return ssl_config_service_manager_->Get();
}

Powered by Google App Engine
This is Rietveld 408576698