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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc

Issue 2334623003: Store net::ProxyServer in HttpResponseInfo object (Closed)
Patch Set: Rebased, fix compile error Created 4 years, 2 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_delegate.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc
index ed2397f1a2c7aeba9bcdfbbf43ed90d5ce28eac0..b6b8f379ce37dca3bb79d6e9b0397783a82315e6 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc
@@ -64,7 +64,7 @@ void DataReductionProxyDelegate::OnTunnelConnectCompleted(
void DataReductionProxyDelegate::OnFallback(const net::ProxyServer& bad_proxy,
int net_error) {
if (bad_proxy.is_valid() &&
- config_->IsDataReductionProxy(bad_proxy.host_port_pair(), nullptr)) {
+ config_->IsDataReductionProxy(bad_proxy, nullptr)) {
event_creator_->AddProxyFallbackEvent(net_log_, bad_proxy.ToURI(),
net_error);
}
@@ -85,8 +85,7 @@ bool DataReductionProxyDelegate::IsTrustedSpdyProxy(
!proxy_server.is_valid()) {
return false;
}
- return config_ &&
- config_->IsDataReductionProxy(proxy_server.host_port_pair(), nullptr);
+ return config_ && config_->IsDataReductionProxy(proxy_server, nullptr);
}
void DataReductionProxyDelegate::OnTunnelHeadersReceived(
@@ -113,8 +112,7 @@ void DataReductionProxyDelegate::GetAlternativeProxy(
return;
if (!config_ ||
- !config_->IsDataReductionProxy(resolved_proxy_server.host_port_pair(),
- nullptr)) {
+ !config_->IsDataReductionProxy(resolved_proxy_server, nullptr)) {
return;
}
@@ -170,8 +168,7 @@ void OnResolveProxyHandler(const GURL& url,
net::ProxyInfo* result) {
DCHECK(config);
DCHECK(result->is_empty() || result->is_direct() ||
- !config->IsDataReductionProxy(result->proxy_server().host_port_pair(),
- NULL));
+ !config->IsDataReductionProxy(result->proxy_server(), NULL));
if (!util::EligibleForDataReductionProxy(*result, url, method))
return;
net::ProxyInfo data_reduction_proxy_info;

Powered by Google App Engine
This is Rietveld 408576698