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

Unified Diff: components/data_reduction_proxy/browser/data_reduction_proxy_protocol.cc

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 | « components/data_reduction_proxy/browser/data_reduction_proxy_params.cc ('k') | no next file » | 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_protocol.cc
diff --git a/components/data_reduction_proxy/browser/data_reduction_proxy_protocol.cc b/components/data_reduction_proxy/browser/data_reduction_proxy_protocol.cc
index 7d362f259f8fd945fd40ff9746e42224053b60c1..565621d6af42db561f6a429383c5aabe4f60860a 100644
--- a/components/data_reduction_proxy/browser/data_reduction_proxy_protocol.cc
+++ b/components/data_reduction_proxy/browser/data_reduction_proxy_protocol.cc
@@ -73,6 +73,15 @@ bool MaybeBypassProxyAndPrepareToRetry(
DataReductionProxyBypassType bypass_type =
GetDataReductionProxyBypassType(original_response_headers,
&data_reduction_proxy_info);
+
+ if (bypass_type == BYPASS_EVENT_TYPE_MISSING_VIA_HEADER_OTHER &&
+ DataReductionProxyParams::
+ IsIncludedInRemoveMissingViaHeaderOtherBypassFieldTrial()) {
+ // Ignore MISSING_VIA_HEADER_OTHER proxy bypass events if the client is part
+ // of the field trial to remove these kinds of bypasses.
+ bypass_type = BYPASS_EVENT_TYPE_MAX;
+ }
+
if (proxy_bypass_type)
*proxy_bypass_type = bypass_type;
if (bypass_type == BYPASS_EVENT_TYPE_MAX)
« no previous file with comments | « components/data_reduction_proxy/browser/data_reduction_proxy_params.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698