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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service.h

Issue 778463002: Wrapped data reduction proxy initialization into its own class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@network-delegate
Patch Set: Addressed comment from mmenke Created 6 years 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: components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service.h
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service.h b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service.h
index 49e56c4d1f20d2d68d7bd11f8252e599cc738940..e639b6ad7c5728913be6fe8227e043b6e7002735 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service.h
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service.h
@@ -101,6 +101,7 @@ class DataReductionProxyConfigTracker : public DataReductionProxyConfigurator {
void Disable() override;
void AddHostPatternToBypass(const std::string& pattern) override;
void AddURLPatternToBypass(const std::string& pattern) override;
+ const net::ProxyConfig& GetProxyConfigOnIOThread() const override;
private:
FRIEND_TEST_ALL_PREFIXES(DataReductionProxyConfigServiceTest,
@@ -114,6 +115,7 @@ class DataReductionProxyConfigTracker : public DataReductionProxyConfigurator {
const net::ProxyConfig& config);
base::Callback<void(bool, const net::ProxyConfig&)> update_proxy_config_;
+ net::ProxyConfig config_;
std::vector<std::string> bypass_rules_;
scoped_refptr<base::TaskRunner> task_runner_;

Powered by Google App Engine
This is Rietveld 408576698