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

Unified Diff: components/network_session_configurator/network_session_configurator.cc

Issue 2850653002: Enable NetProxyPreconnects experiment by default (Closed)
Patch Set: 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 | « no previous file | net/http/http_network_session.h » ('j') | net/http/http_stream_factory_impl.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/network_session_configurator/network_session_configurator.cc
diff --git a/components/network_session_configurator/network_session_configurator.cc b/components/network_session_configurator/network_session_configurator.cc
index da1537cf7bfb45b3cb42b6af513e929cddccb666..8a9d94099e4cda076b766d94e505c4447d493427 100644
--- a/components/network_session_configurator/network_session_configurator.cc
+++ b/components/network_session_configurator/network_session_configurator.cc
@@ -321,14 +321,6 @@ void ConfigureQuicParams(base::StringPiece quic_trial_group,
}
}
-void ConfigureOptimizePreconnectsToProxiesParams(
- const std::map<std::string, std::string>& proxy_preconnects_trial_params,
- net::HttpNetworkSession::Params* params) {
- params->restrict_to_one_preconnect_for_proxies =
- GetVariationParam(proxy_preconnects_trial_params,
- "restrict_to_one_preconnect_for_proxies") == "true";
-}
-
} // anonymous namespace
namespace network_session_configurator {
@@ -373,8 +365,6 @@ void ParseFieldTrials(bool is_quic_force_disabled,
std::map<std::string, std::string> proxy_preconnects_trial_params;
variations::GetVariationParams("NetProxyPreconnects",
&proxy_preconnects_trial_params);
xunjieli 2017/04/28 14:24:30 Can we get rid of line 365 - 367?
tbansal1 2017/04/28 16:36:08 Good catch. Done.
- ConfigureOptimizePreconnectsToProxiesParams(proxy_preconnects_trial_params,
- params);
}
} // namespace network_session_configurator
« no previous file with comments | « no previous file | net/http/http_network_session.h » ('j') | net/http/http_stream_factory_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698