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

Unified Diff: components/data_reduction_proxy/browser/data_reduction_proxy_params.h

Issue 583363002: Ignore MISSING_VIA_HEADER_OTHER bypasses if in relevant finch trial (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
« no previous file with comments | « no previous file | components/data_reduction_proxy/browser/data_reduction_proxy_params.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/data_reduction_proxy/browser/data_reduction_proxy_params.h
diff --git a/components/data_reduction_proxy/browser/data_reduction_proxy_params.h b/components/data_reduction_proxy/browser/data_reduction_proxy_params.h
index f9ec3a308c7e1da00ae33ad9a3cd18d1bedf2d47..a4fce08a9eb5c22c76fb223d0f6113f80cd57cc3 100644
--- a/components/data_reduction_proxy/browser/data_reduction_proxy_params.h
+++ b/components/data_reduction_proxy/browser/data_reduction_proxy_params.h
@@ -83,6 +83,13 @@ class DataReductionProxyParams {
// is in effect.
static bool IsIncludedInHoldbackFieldTrial();
+ // Returns true if this client is part of a field trial that removes the
+ // |MISSING_VIA_HEADER_OTHER| proxy bypass case. This experiment changes proxy
+ // bypass logic to not trigger a proxy bypass when a response with a non-4xx
+ // response code is expected to have a data reduction proxy via header, but
+ // the data reduction proxy via header is missing.
+ static bool IsIncludedInRemoveMissingViaHeaderOtherBypassFieldTrial();
+
// 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
« no previous file with comments | « no previous file | components/data_reduction_proxy/browser/data_reduction_proxy_params.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698