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

Unified Diff: components/data_reduction_proxy/core/common/data_reduction_proxy_params.h

Issue 903213003: Enable QUIC for proxies based on Finch config and command line switch. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed formatting Created 5 years, 10 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/common/data_reduction_proxy_params.h
diff --git a/components/data_reduction_proxy/core/common/data_reduction_proxy_params.h b/components/data_reduction_proxy/core/common/data_reduction_proxy_params.h
index e8164d7a6f767c5bd7467bbbddfdcaff755e4ccf..c574d3eb48dd39c644a015fd7e9b45ca3d8a3a70 100644
--- a/components/data_reduction_proxy/core/common/data_reduction_proxy_params.h
+++ b/components/data_reduction_proxy/core/common/data_reduction_proxy_params.h
@@ -99,6 +99,16 @@ class DataReductionProxyParams {
static bool IsIncludedInAndroidOnePromoFieldTrial(
const char* build_fingerprint);
+ // Returns true if this client is part of a field trial that sets the origin
+ // proxy server as quic://proxy.googlezip.net.
+ static bool IsIncludedInUseQUICFieldTrial();
+
+ static std::string GetFieldTrialName();
+
+ // Sets if QUIC is enabled for proxies. Also, may update the origin proxy
bengr 2015/02/11 23:57:28 What do you mean by 'may'? Can't you just say: //
tbansal1 2015/02/12 02:27:22 Done.
+ // server to a QUIC server.
+ void EnableQUIC(bool quic_enabled_for_proxies);
+
// Constructs configuration parameters. If |kAllowed|, then the standard
// data reduction proxy configuration is allowed to be used. If
// |kfallbackAllowed| a fallback proxy can be used if the primary proxy is
@@ -260,6 +270,11 @@ class DataReductionProxyParams {
return configured_on_command_line_;
}
+ // Returns true if QUIC is enabled for proxies.
+ bool IsQUICEnabled() const {
bengr 2015/02/11 23:57:28 IsQuicEnabled
tbansal1 2015/02/12 02:27:22 Done.
+ return quic_enabled_for_proxies_;
+ }
+
protected:
// Test constructor that optionally won't call Init();
DataReductionProxyParams(int flags,
@@ -321,6 +336,8 @@ class DataReductionProxyParams {
bool alt_fallback_allowed_;
bool promo_allowed_;
bool holdback_;
+ bool quic_enabled_for_proxies_;
bengr 2015/02/11 23:57:28 quic_enabled_
tbansal1 2015/02/12 02:27:22 Done.
+ std::string command_line_origin_;
bool configured_on_command_line_;
};

Powered by Google App Engine
This is Rietveld 408576698