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

Unified Diff: android_webview/browser/aw_browser_context.cc

Issue 792803007: Make Data Reduction Proxy a best effort proxy (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: updated tests Created 5 years, 11 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 016f35e30e6366be7943585da64800006ec944b5..1fea722a29f1ffb92ef83aba90d1711f5bfe650c 100644
--- a/android_webview/browser/aw_browser_context.cc
+++ b/android_webview/browser/aw_browser_context.cc
@@ -18,7 +18,7 @@
#include "base/prefs/pref_service.h"
#include "base/prefs/pref_service_factory.h"
#include "components/autofill/core/common/autofill_pref_names.h"
-#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service.h"
+#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator.h"
#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_prefs.h"
#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.h"
#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_statistics_prefs.h"
@@ -36,7 +36,7 @@
using base::FilePath;
using content::BrowserThread;
-using data_reduction_proxy::DataReductionProxyConfigService;
+using data_reduction_proxy::DataReductionProxyConfigurator;
using data_reduction_proxy::DataReductionProxyEventStore;
using data_reduction_proxy::DataReductionProxySettings;
@@ -135,29 +135,6 @@ void AwBrowserContext::SetLegacyCacheRemovalDelayForTest(int delay_ms) {
void AwBrowserContext::PreMainMessageLoopRun() {
cookie_store_ = CreateCookieStore(this);
- data_reduction_proxy_settings_.reset(
- new DataReductionProxySettings(
- new data_reduction_proxy::DataReductionProxyParams(
- data_reduction_proxy::DataReductionProxyParams::kAllowed)));
- data_reduction_proxy_event_store_.reset(
- new DataReductionProxyEventStore(
- BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI)));
- scoped_ptr<DataReductionProxyConfigService>
- data_reduction_proxy_config_service(
- new DataReductionProxyConfigService(
- scoped_ptr<net::ProxyConfigService>(
- CreateProxyConfigService()).Pass()));
- if (data_reduction_proxy_settings_.get()) {
- data_reduction_proxy_configurator_.reset(
- new data_reduction_proxy::DataReductionProxyConfigTracker(
- base::Bind(&DataReductionProxyConfigService::UpdateProxyConfig,
- base::Unretained(
- data_reduction_proxy_config_service.get())),
- BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO)));
- data_reduction_proxy_settings_->SetProxyConfigurator(
- data_reduction_proxy_configurator_.get());
- }
-
FilePath cache_path;
const FilePath fallback_cache_dir =
GetPath().Append(FILE_PATH_LITERAL("Cache"));
@@ -175,9 +152,24 @@ void AwBrowserContext::PreMainMessageLoopRun() {
<< "Using app data directory as a fallback.";
}
url_request_context_getter_ =
- new AwURLRequestContextGetter(cache_path,
- cookie_store_.get(),
- data_reduction_proxy_config_service.Pass());
+ new AwURLRequestContextGetter(
+ cache_path, cookie_store_.get(),
+ make_scoped_ptr(CreateProxyConfigService()).Pass());
+
+ data_reduction_proxy_settings_.reset(
+ new DataReductionProxySettings(
+ new data_reduction_proxy::DataReductionProxyParams(
+ data_reduction_proxy::DataReductionProxyParams::kAllowed)));
+ data_reduction_proxy_event_store_.reset(
+ new DataReductionProxyEventStore(
+ BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI)));
+ data_reduction_proxy_configurator_.reset(
+ new data_reduction_proxy::DataReductionProxyConfigurator(
+ BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO),
+ url_request_context_getter_->GetNetLog(),
+ data_reduction_proxy_event_store_.get()));
+ data_reduction_proxy_settings_->SetProxyConfigurator(
+ data_reduction_proxy_configurator_.get());
visitedlink_master_.reset(
new visitedlink::VisitedLinkMaster(this, this, false));
@@ -236,6 +228,11 @@ DataReductionProxyEventStore*
return data_reduction_proxy_event_store_.get();
}
+data_reduction_proxy::DataReductionProxyConfigurator*
+AwBrowserContext::GetDataReductionProxyConfigurator() {
+ return data_reduction_proxy_configurator_.get();
+}
+
AwURLRequestContextGetter* AwBrowserContext::GetAwURLRequestContext() {
return url_request_context_getter_.get();
}
« no previous file with comments | « android_webview/browser/aw_browser_context.h ('k') | android_webview/browser/net/aw_url_request_context_getter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698