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

Unified Diff: android_webview/browser/aw_browser_context.cc

Issue 473723002: Update data reduction proxy statistics prefs less often on desktop (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@tooManyWritesPatch
Patch Set: Addressed bengr comments Created 6 years, 4 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: android_webview/browser/aw_browser_context.cc
diff --git a/android_webview/browser/aw_browser_context.cc b/android_webview/browser/aw_browser_context.cc
index b188efcd50590a2b1a393f4c9b8ccc2392262831..e2fcb2213845cdcb61fecc90bceeef9b2c65dacc 100644
--- a/android_webview/browser/aw_browser_context.cc
+++ b/android_webview/browser/aw_browser_context.cc
@@ -20,6 +20,7 @@
#include "components/data_reduction_proxy/browser/data_reduction_proxy_params.h"
#include "components/data_reduction_proxy/browser/data_reduction_proxy_prefs.h"
#include "components/data_reduction_proxy/browser/data_reduction_proxy_settings.h"
+#include "components/data_reduction_proxy/browser/data_reduction_proxy_statistics_prefs.h"
#include "components/user_prefs/user_prefs.h"
#include "components/visitedlink/browser/visitedlink_master.h"
#include "content/public/browser/browser_thread.h"
@@ -33,6 +34,7 @@ using base::FilePath;
using content::BrowserThread;
using data_reduction_proxy::DataReductionProxyConfigService;
using data_reduction_proxy::DataReductionProxySettings;
+using data_reduction_proxy::DataReductionProxyStatisticsPrefs;
namespace android_webview {
@@ -104,6 +106,11 @@ void AwBrowserContext::PreMainMessageLoopRun() {
new DataReductionProxySettings(
new data_reduction_proxy::DataReductionProxyParams(
data_reduction_proxy::DataReductionProxyParams::kAllowed)));
+ data_reduction_proxy_statistics_prefs_.reset(
+ new DataReductionProxyStatisticsPrefs(
+ g_browser_process->local_state(),
sgurun-gerrit only 2014/09/03 01:00:38 does this even compile? g_browser_process is a bro
megjablon 2014/09/03 18:58:30 My bad. It looks like we're using user_pref_servic
+ base::MessageLoopProxy::current(),
+ base::TimeDelta()));
#endif
scoped_ptr<DataReductionProxyConfigService>
data_reduction_proxy_config_service(
@@ -209,9 +216,11 @@ void AwBrowserContext::CreateUserPrefServiceIfNecessary() {
if (data_reduction_proxy_settings_.get()) {
data_reduction_proxy_settings_->InitDataReductionProxySettings(
user_pref_service_.get(),
- user_pref_service_.get(),
GetRequestContext());
+ data_reduction_proxy_settings_->SetDataReductionProxyStatisticsPrefs(
+ data_reduction_proxy_statistics_prefs_.get());
+
data_reduction_proxy_settings_->SetDataReductionProxyEnabled(
data_reduction_proxy_enabled_);
}

Powered by Google App Engine
This is Rietveld 408576698