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

Unified Diff: net/http/http_stream_factory_impl.cc

Issue 2850653002: Enable NetProxyPreconnects experiment by default (Closed)
Patch Set: xunjieli comments Created 3 years, 8 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 | « net/http/http_network_session.cc ('k') | net/http/http_stream_factory_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_stream_factory_impl.cc
diff --git a/net/http/http_stream_factory_impl.cc b/net/http/http_stream_factory_impl.cc
index 48bfd1ddbf6f9d87ad0bd5a3cc3de8d4557795c6..25799331eb688be94fe6e8656442ae253e10ce86 100644
--- a/net/http/http_stream_factory_impl.cc
+++ b/net/http/http_stream_factory_impl.cc
@@ -300,10 +300,8 @@ bool HttpStreamFactoryImpl::OnInitConnection(const JobController& controller,
return false;
}
- if (!session_->params().restrict_to_one_preconnect_for_proxies ||
- !ProxyServerSupportsPriorities(proxy_info)) {
+ if (!ProxyServerSupportsPriorities(proxy_info))
return false;
- }
PreconnectingProxyServer preconnecting_proxy_server(proxy_info.proxy_server(),
privacy_mode);
@@ -312,7 +310,7 @@ bool HttpStreamFactoryImpl::OnInitConnection(const JobController& controller,
preconnecting_proxy_server)) {
UMA_HISTOGRAM_EXACT_LINEAR("Net.PreconnectSkippedToProxyServers", 1, 2);
// Skip preconnect to the proxy server since we are already preconnecting
- // (probably via some other job).
+ // (probably via some other job). See crbug.com/682041 for details.
return true;
}
« no previous file with comments | « net/http/http_network_session.cc ('k') | net/http/http_stream_factory_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698