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

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

Issue 2334623003: Store net::ProxyServer in HttpResponseInfo object (Closed)
Patch Set: PS Created 4 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
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 8f86b0cca2f82d3078929d41d9b7497048585878..13d159df28c22dcb30d2bf02b90a9a37d3763e5f 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
@@ -65,7 +65,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);
}
@@ -86,8 +86,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(
@@ -114,8 +113,7 @@ void DataReductionProxyDelegate::GetAlternativeProxy(
return;
if (!config_ ||
- !config_->IsDataReductionProxy(resolved_proxy_server.host_port_pair(),
- nullptr)) {
+ !config_->IsDataReductionProxy(resolved_proxy_server, nullptr)) {
return;
}
@@ -157,8 +155,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