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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.h

Issue 836583002: Add support for field trial group to relax missing via header bypass logic. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unused include from aw_url_request_context_getter.cc Created 5 years, 11 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/browser/data_reduction_proxy_network_delegate.h
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.h b/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.h
index 90398b25f862783a8985ec764d9eeb6b1801c402..0744b86c466fe82822b6ef2d63c7a9e5cedf7a1b 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.h
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.h
@@ -51,6 +51,7 @@ class DataReductionProxyNetworkDelegate : public net::LayeredNetworkDelegate {
// configuration is provided along with the resolution for this URL, in
// |result|, whch may be modified. Retry info is presumed to be from the proxy
// service.
+ // TODO(sclittle): Remove this, see http://crbug.com/447346.
typedef base::Callback<void(
const GURL& url,
int load_flags,
@@ -158,6 +159,7 @@ class DataReductionProxyNetworkDelegate : public net::LayeredNetworkDelegate {
DataReductionProxyStatisticsPrefs* data_reduction_proxy_statistics_prefs_;
+ // TODO(sclittle): Factor this out, see http://crbug.com/447346.
OnResolveProxyHandler on_resolve_proxy_handler_;
ProxyConfigGetter proxy_config_getter_;
@@ -165,6 +167,19 @@ class DataReductionProxyNetworkDelegate : public net::LayeredNetworkDelegate {
DISALLOW_COPY_AND_ASSIGN(DataReductionProxyNetworkDelegate);
};
+// Adds data reduction proxies to |result|, where applicable, if result
+// otherwise uses a direct connection for |url|, and the data reduction proxy is
+// not bypassed. Also, configures |result| to proceed directly to the origin if
+// |result|'s current proxy is the data 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 net::ProxyConfig& data_reduction_proxy_config,
+ const net::ProxyRetryInfoMap& proxy_retry_info,
+ const DataReductionProxyParams* params,
+ net::ProxyInfo* result);
+
} // namespace data_reduction_proxy
#endif // COMPONENTS_DATA_REDUCTION_PROXY_CORE_BROWSER_DATA_REDUCTION_PROXY_NETWORK_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698