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

Side by Side Diff: components/network_session_configurator/network_session_configurator.cc

Issue 2283393002: Use a field trial to decide if QUIC should attempt to prevent packet fragmentation. (Closed)
Patch Set: Rebase 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 unified diff | Download patch
« no previous file with comments | « no previous file | components/network_session_configurator/network_session_configurator_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/network_session_configurator/network_session_configurator.h " 5 #include "components/network_session_configurator/network_session_configurator.h "
6 6
7 #include <map> 7 #include <map>
8 8
9 #include "base/metrics/field_trial.h" 9 #include "base/metrics/field_trial.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 208 matching lines...) Expand 10 before | Expand all | Expand 10 after
219 return 0; 219 return 0;
220 } 220 }
221 221
222 bool ShouldQuicRaceCertVerification( 222 bool ShouldQuicRaceCertVerification(
223 const VariationParameters& quic_trial_params) { 223 const VariationParameters& quic_trial_params) {
224 return base::LowerCaseEqualsASCII( 224 return base::LowerCaseEqualsASCII(
225 GetVariationParam(quic_trial_params, "race_cert_verification"), 225 GetVariationParam(quic_trial_params, "race_cert_verification"),
226 "true"); 226 "true");
227 } 227 }
228 228
229 bool ShouldQuicDoNotFragment(
230 const VariationParameters& quic_trial_params) {
231 return base::LowerCaseEqualsASCII(
232 GetVariationParam(quic_trial_params, "do_not_fragment"),
233 "true");
234 }
235
229 bool ShouldQuicDisablePreConnectIfZeroRtt( 236 bool ShouldQuicDisablePreConnectIfZeroRtt(
230 const VariationParameters& quic_trial_params) { 237 const VariationParameters& quic_trial_params) {
231 return base::LowerCaseEqualsASCII( 238 return base::LowerCaseEqualsASCII(
232 GetVariationParam(quic_trial_params, "disable_preconnect_if_0rtt"), 239 GetVariationParam(quic_trial_params, "disable_preconnect_if_0rtt"),
233 "true"); 240 "true");
234 } 241 }
235 242
236 std::unordered_set<std::string> GetQuicHostWhitelist( 243 std::unordered_set<std::string> GetQuicHostWhitelist(
237 const VariationParameters& quic_trial_params) { 244 const VariationParameters& quic_trial_params) {
238 std::string whitelist = 245 std::string whitelist =
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
341 idle_connection_timeout_seconds; 348 idle_connection_timeout_seconds;
342 } 349 }
343 int packet_reader_yield_after_duration_milliseconds = 350 int packet_reader_yield_after_duration_milliseconds =
344 GetQuicPacketReaderYieldAfterDurationMilliseconds(quic_trial_params); 351 GetQuicPacketReaderYieldAfterDurationMilliseconds(quic_trial_params);
345 if (packet_reader_yield_after_duration_milliseconds != 0) { 352 if (packet_reader_yield_after_duration_milliseconds != 0) {
346 params->quic_packet_reader_yield_after_duration_milliseconds = 353 params->quic_packet_reader_yield_after_duration_milliseconds =
347 packet_reader_yield_after_duration_milliseconds; 354 packet_reader_yield_after_duration_milliseconds;
348 } 355 }
349 params->quic_race_cert_verification = 356 params->quic_race_cert_verification =
350 ShouldQuicRaceCertVerification(quic_trial_params); 357 ShouldQuicRaceCertVerification(quic_trial_params);
358 params->quic_do_not_fragment =
359 ShouldQuicDoNotFragment(quic_trial_params);
351 params->quic_disable_preconnect_if_0rtt = 360 params->quic_disable_preconnect_if_0rtt =
352 ShouldQuicDisablePreConnectIfZeroRtt(quic_trial_params); 361 ShouldQuicDisablePreConnectIfZeroRtt(quic_trial_params);
353 params->quic_host_whitelist = GetQuicHostWhitelist(quic_trial_params); 362 params->quic_host_whitelist = GetQuicHostWhitelist(quic_trial_params);
354 params->quic_migrate_sessions_on_network_change = 363 params->quic_migrate_sessions_on_network_change =
355 ShouldQuicMigrateSessionsOnNetworkChange(quic_trial_params); 364 ShouldQuicMigrateSessionsOnNetworkChange(quic_trial_params);
356 params->quic_migrate_sessions_early = 365 params->quic_migrate_sessions_early =
357 ShouldQuicMigrateSessionsEarly(quic_trial_params); 366 ShouldQuicMigrateSessionsEarly(quic_trial_params);
358 params->quic_allow_server_migration = 367 params->quic_allow_server_migration =
359 ShouldQuicAllowServerMigration(quic_trial_params); 368 ShouldQuicAllowServerMigration(quic_trial_params);
360 } 369 }
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
406 std::string http2_trial_group = 415 std::string http2_trial_group =
407 base::FieldTrialList::FindFullName(kHttp2FieldTrialName); 416 base::FieldTrialList::FindFullName(kHttp2FieldTrialName);
408 ConfigureHttp2Params(http2_trial_group, params); 417 ConfigureHttp2Params(http2_trial_group, params);
409 418
410 const std::string tfo_trial_group = 419 const std::string tfo_trial_group =
411 base::FieldTrialList::FindFullName(kTCPFastOpenFieldTrialName); 420 base::FieldTrialList::FindFullName(kTCPFastOpenFieldTrialName);
412 ConfigureTCPFastOpenParams(tfo_trial_group, params); 421 ConfigureTCPFastOpenParams(tfo_trial_group, params);
413 } 422 }
414 423
415 } // namespace network_session_configurator 424 } // namespace network_session_configurator
OLDNEW
« no previous file with comments | « no previous file | components/network_session_configurator/network_session_configurator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698