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

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

Issue 2581313002: Cleanup DataReductionProxy (DRP) Brotli code (Closed)
Patch Set: Fix thread checker Created 4 years 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 9161b87a5063cbc366c85d71d702d44dd794bd9e..b18dee444721063cb4a29b2909deddbdc33d3b14 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
@@ -52,7 +52,7 @@ class DataReductionProxyRequestOptions {
// Sets |key_| to the default key and initializes the credentials, version,
// client, and lo-fi header values. Generates the |header_value_| string,
- // which is concatenated to the Chrome-proxy header.
+ // which is concatenated to the Chrome-proxy header. Called on the UI thread.
void Init();
// Adds a 'Chrome-Proxy' header to |request_headers| with the data reduction
@@ -98,6 +98,10 @@ class DataReductionProxyRequestOptions {
const std::string& version,
DataReductionProxyConfig* config);
+ // Returns the chrome proxy header. Protected so that it is available for
+ // testing.
+ std::string GetHeaderValueForTesting() const;
+
private:
FRIEND_TEST_ALL_PREFIXES(DataReductionProxyRequestOptionsTest,
AuthHashForSalt);

Powered by Google App Engine
This is Rietveld 408576698