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

Unified Diff: ios/chrome/browser/ios_chrome_io_thread.mm

Issue 2019933002: Enable AltSvc pooling by default. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2743
Patch Set: Created 4 years, 7 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: ios/chrome/browser/ios_chrome_io_thread.mm
diff --git a/ios/chrome/browser/ios_chrome_io_thread.mm b/ios/chrome/browser/ios_chrome_io_thread.mm
index a85c5656de9ecd4b7631de1aab82926488488ef3..3dcbbad8fa316eb9909993e0cb223ecc6ce119ff 100644
--- a/ios/chrome/browser/ios_chrome_io_thread.mm
+++ b/ios/chrome/browser/ios_chrome_io_thread.mm
@@ -752,12 +752,8 @@ void IOSChromeIOThread::ConfigureQuicGlobals(
bool enable_quic = ShouldEnableQuic(quic_trial_group);
globals->enable_quic.set(enable_quic);
- if (ShouldQuicEnableAlternativeServicesForDifferentHost(quic_trial_params)) {
- globals->enable_alternative_service_with_different_host.set(true);
- globals->parse_alternative_services.set(true);
- } else {
- globals->enable_alternative_service_with_different_host.set(false);
- }
+ globals->enable_alternative_service_with_different_host.set(
+ ShouldQuicEnableAlternativeServicesForDifferentHost(quic_trial_params));
if (enable_quic) {
globals->quic_always_require_handshake_confirmation.set(
@@ -887,15 +883,10 @@ bool IOSChromeIOThread::ShouldQuicPreferAes(
bool IOSChromeIOThread::ShouldQuicEnableAlternativeServicesForDifferentHost(
const VariationParameters& quic_trial_params) {
- // TODO(bnc): Remove inaccurately named "use_alternative_services" parameter.
- return base::LowerCaseEqualsASCII(
- GetVariationParam(quic_trial_params, "use_alternative_services"),
- "true") ||
- base::LowerCaseEqualsASCII(
- GetVariationParam(
- quic_trial_params,
- "enable_alternative_service_with_different_host"),
- "true");
+ return !base::LowerCaseEqualsASCII(
+ GetVariationParam(quic_trial_params,
+ "enable_alternative_service_with_different_host"),
+ "false");
}
int IOSChromeIOThread::GetQuicMaxNumberOfLossyConnections(

Powered by Google App Engine
This is Rietveld 408576698