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

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

Issue 1363673004: [DRP] Consistently use LoFi for an entire page (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressing tbansal comments Created 5 years, 3 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 240eed6983cc24ad9c3d434ea22cc27f30633dac..8134fb170c1e9008c7ce1922ef32e5062bcdf192 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
@@ -98,7 +98,8 @@ class DataReductionProxyRequestOptions {
// CONNECT server. Must be called on the IO thread.
void MaybeAddRequestHeader(net::URLRequest* request,
const net::ProxyServer& proxy_server,
- net::HttpRequestHeaders* request_headers);
+ net::HttpRequestHeaders* request_headers,
+ bool is_lofi_on);
// Adds a 'Chrome-Proxy' header to |request_headers| with the data reduction
// proxy authentication credentials. Only adds this header if the provided
@@ -136,7 +137,8 @@ class DataReductionProxyRequestOptions {
protected:
void SetHeader(const net::URLRequest* request,
- net::HttpRequestHeaders* headers);
+ net::HttpRequestHeaders* headers,
+ bool is_lofi_on);
// Returns a UTF16 string that's the hash of the configured authentication
// |key| and |salt|. Returns an empty UTF16 string if no key is configured or
@@ -172,7 +174,7 @@ class DataReductionProxyRequestOptions {
void UpdateVersion();
// May regenerate the Chrome Proxy header based on changes in Lo-Fi status.
- void MayRegenerateHeaderBasedOnLoFi(const net::URLRequest* request);
+ void MayRegenerateHeaderBasedOnLoFi(bool is_lofi_on);
// Update the value of the experiments to be run and regenerate the header if
// necessary.
@@ -194,7 +196,8 @@ class DataReductionProxyRequestOptions {
void MaybeAddRequestHeaderImpl(const net::URLRequest* request,
const net::HostPortPair& proxy_server,
bool expect_ssl,
- net::HttpRequestHeaders* request_headers);
+ net::HttpRequestHeaders* request_headers,
+ bool is_lofi_on);
// Regenerates the |header_value_| string which is concatenated to the
// Chrome-proxy header.

Powered by Google App Engine
This is Rietveld 408576698