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

Unified Diff: trunk/src/components/data_reduction_proxy/browser/data_reduction_proxy_config_service.cc

Issue 307013003: Revert 273810 "Added alternative configuration for the data redu..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 7 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: trunk/src/components/data_reduction_proxy/browser/data_reduction_proxy_config_service.cc
===================================================================
--- trunk/src/components/data_reduction_proxy/browser/data_reduction_proxy_config_service.cc (revision 273823)
+++ trunk/src/components/data_reduction_proxy/browser/data_reduction_proxy_config_service.cc (working copy)
@@ -130,8 +130,7 @@
bool primary_restricted,
bool fallback_restricted,
const std::string& primary_origin,
- const std::string& fallback_origin,
- const std::string& ssl_origin) {
+ const std::string& fallback_origin) {
std::vector<std::string> proxies;
if (!primary_restricted) {
@@ -152,14 +151,9 @@
return;
}
- std::string trimmed_ssl;
- base::TrimString(ssl_origin, "/", &trimmed_ssl);
-
- std::string server = "http=" + JoinString(proxies, ",") + ",direct://;"
- + (ssl_origin.empty() ? "" : ("https=" + ssl_origin + ",direct://;"));
-
net::ProxyConfig config;
- config.proxy_rules().ParseFromString(server);
+ config.proxy_rules().ParseFromString(
+ "http=" + JoinString(proxies, ",") + ",direct://;");
config.proxy_rules().bypass_rules.ParseFromString(
JoinString(bypass_rules_, ", "));
UpdateProxyConfigOnIOThread(true, config);

Powered by Google App Engine
This is Rietveld 408576698