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

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

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_config.h
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h
index e48c9184d1c7b5e6ede56236cf596837af4c831d..22040a2ed440ae775567252fec39b8ec006dcf97 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h
@@ -117,9 +117,8 @@ class DataReductionProxyConfig
// proxy that was used. Subsequent entries in |proxy_info.proxy_servers| will
// contain the names of the Data Reduction Proxy servers that would be used if
// |proxy_info.proxy_servers.front()| is bypassed, if any exist. In addition,
- // |proxy_info| will note if the proxy used was a fallback, or a proxy for
- // ssl; these are not mutually exclusive. |proxy_info| can be NULL if the
- // caller isn't interested in its values.
+ // |proxy_info| will note if the proxy used was a fallback. |proxy_info| can
+ // be NULL if the caller isn't interested in its values.
virtual bool WasDataReductionProxyUsed(
const net::URLRequest* request,
DataReductionProxyTypeInfo* proxy_info) const;
@@ -129,10 +128,9 @@ class DataReductionProxyConfig
// the proxy that matches. Subsequent entries in |proxy_info.proxy_servers|
// will contain the name of the Data Reduction Proxy servers that would be
// used if |proxy_info.proxy_servers.front()| is bypassed, if any exist. In
- // addition, |proxy_info| will note if the proxy was a fallback or a proxy for
- // ssl; these are not mutually exclusive. |proxy_info| can be NULL if the
- // caller isn't interested in its values.
- // Virtual for testing.
+ // addition, |proxy_info| will note if the proxy was a fallback. |proxy_info|
+ // can be NULL if the caller isn't interested in its values. Virtual for
+ // testing.
virtual bool IsDataReductionProxy(
const net::HostPortPair& host_port_pair,
DataReductionProxyTypeInfo* proxy_info) const;
@@ -169,9 +167,6 @@ class DataReductionProxyConfig
virtual bool ContainsDataReductionProxy(
const net::ProxyConfig::ProxyRules& proxy_rules) const;
- // Returns true if the proxy was using the HTTP tunnel for a HTTPS origin.
- bool UsingHTTPTunnel(const net::HostPortPair& proxy_server) const;
-
// Returns true if the Data Reduction Proxy configuration may be used.
bool allowed() const;

Powered by Google App Engine
This is Rietveld 408576698