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

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: Addressed comments and removed code duplication. 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..01b73a9eeb3f8a969fb8c4136e0e62da2aef9693 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,14 @@ 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();
+
+ // Sets if QUIC is enabled for proxies. Also, may update the origin proxy
+ // server to a QUIC server.
+ void SetIsQuicEnabledForProxies(bool quic_enabled_for_proxies);
bengr 2015/02/10 20:18:31 void EnableQUIC(bool enable)
tbansal1 2015/02/10 22:24:39 Done.
+
// 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 +268,11 @@ class DataReductionProxyParams {
return configured_on_command_line_;
}
+ // Returns true if QUIC is enabled for proxies.
+ bool is_quic_enabled_for_proxies_() const {
bengr 2015/02/10 20:18:31 bool IsQUICEnabled() const;
tbansal1 2015/02/10 22:24:39 Done.
+ return quic_enabled_for_proxies_;
+ }
+
protected:
// Test constructor that optionally won't call Init();
DataReductionProxyParams(int flags,
@@ -321,6 +334,8 @@ class DataReductionProxyParams {
bool alt_fallback_allowed_;
bool promo_allowed_;
bool holdback_;
+ bool quic_enabled_for_proxies_;
+ std::string command_line_origin_;
bool configured_on_command_line_;
};

Powered by Google App Engine
This is Rietveld 408576698