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

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

Issue 332313003: Add Finch experiment for selectively bypassing proxies. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address bengr's feedback Created 6 years, 6 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/browser/data_reduction_proxy_protocol.h
diff --git a/components/data_reduction_proxy/browser/data_reduction_proxy_protocol.h b/components/data_reduction_proxy/browser/data_reduction_proxy_protocol.h
index 5598c26acc7f34ff35aa7ad5fb8a8bf6fa9da71e..9d3d72c67c2276ed02981691dba50b677f2dfa1c 100644
--- a/components/data_reduction_proxy/browser/data_reduction_proxy_protocol.h
+++ b/components/data_reduction_proxy/browser/data_reduction_proxy_protocol.h
@@ -14,6 +14,7 @@ class TimeDelta;
namespace net {
class HttpResponseHeaders;
+class ProxyInfo;
class ProxyServer;
class URLRequest;
}
@@ -33,6 +34,14 @@ bool MaybeBypassProxyAndPrepareToRetry(
const net::HttpResponseHeaders* original_response_headers,
scoped_refptr<net::HttpResponseHeaders>* override_response_headers);
+// Upon |ProxyService.ResolveProxy()|, bypass if the effective proxy is the data
bengr 2014/07/07 16:28:14 Say something about when this method is called. E.
+// reduction proxy, the |net::LOAD_BYPASS_DATA_REDUCTION_PROXY| |load_flag| is
+// set, and the DataCompressionProxyCriticalBypass Finch trial is set.
+void OnResolveProxyHandler(const GURL& url,
+ int load_flags,
+ const DataReductionProxyParams* params,
+ net::ProxyInfo* result);
+
// Returns true if the request method is idempotent. Only idempotent requests
// are retried on a bypass. Visible as part of the public API for testing.
bool IsRequestIdempotent(const net::URLRequest* request);

Powered by Google App Engine
This is Rietveld 408576698