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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol.cc

Issue 1871783002: Remove the Data Reduction Proxy TLS experiment code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: tbansal comments Created 4 years, 8 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: components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol.cc
index cab7e33380f90cd09e71abbabf14794bcde851ca..1d0349a0e2493849316fa649fc696a3ae8d89d9b 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol.cc
@@ -119,16 +119,10 @@ bool DataReductionProxyBypassProtocol::MaybeBypassProxyAndPrepareToRetry(
data_reduction_proxy_type_info.proxy_servers.push_back(net::ProxyServer(
net::ProxyServer::SCHEME_HTTP, request->proxy_server()));
data_reduction_proxy_type_info.is_fallback = false;
- data_reduction_proxy_type_info.is_ssl =
- request->url().SchemeIsCryptographic();
} else {
ReportResponseProxyServerStatusHistogram(RESPONSE_PROXY_SERVER_STATUS_DRP);
}
- // TODO(bengr): Implement bypass for CONNECT tunnel.
- if (data_reduction_proxy_type_info.is_ssl)
- return false;
-
if (data_reduction_proxy_type_info.proxy_servers.empty())
return false;

Powered by Google App Engine
This is Rietveld 408576698