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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_request_options.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_request_options.h
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_request_options.h b/components/data_reduction_proxy/core/browser/data_reduction_proxy_request_options.h
index 556559610dcae11ae2f093663c270f9631ff30bd..9506f03d24d9644dd9650829ba87a963b5c47295 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_request_options.h
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_request_options.h
@@ -81,19 +81,10 @@ class DataReductionProxyRequestOptions {
// Adds a 'Chrome-Proxy' header to |request_headers| with the data reduction
// proxy authentication credentials. Only adds this header if the
- // provided |proxy_server| is a data reduction proxy and not the data
- // reduction proxy's CONNECT server.
+ // provided |proxy_server| is a data reduction proxy.
void MaybeAddRequestHeader(const net::ProxyServer& proxy_server,
net::HttpRequestHeaders* request_headers);
- // Adds a 'Chrome-Proxy' header to |request_headers| with the data reduction
- // proxy authentication credentials. Only adds this header if the provided
- // |proxy_server| is the data reduction proxy's CONNECT server. Must be called
- // on the IO thread.
- void MaybeAddProxyTunnelRequestHandler(
- const net::HostPortPair& proxy_server,
- net::HttpRequestHeaders* request_headers);
-
// Stores the supplied key and sets up credentials suitable for authenticating
// with the data reduction proxy.
// This can be called more than once. For example on a platform that does not
@@ -155,14 +146,6 @@ class DataReductionProxyRequestOptions {
// Generates and updates the session ID and credentials.
void UpdateCredentials();
- // Adds authentication headers only if |expects_ssl| is true and
- // |proxy_server| is a data reduction proxy used for ssl tunneling via
- // HTTP CONNECT, or |expect_ssl| is false and |proxy_server| is a data
- // reduction proxy for HTTP traffic.
- void MaybeAddRequestHeaderImpl(const net::HostPortPair& proxy_server,
- bool expect_ssl,
- net::HttpRequestHeaders* request_headers);
-
// Regenerates the |header_value_| string which is concatenated to the
// Chrome-proxy header.
void RegenerateRequestHeaderValue();

Powered by Google App Engine
This is Rietveld 408576698