diff options
author | rtenneti <rtenneti@chromium.org> | 2014-09-29 11:16:08 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-09-29 18:16:47 +0000 |
commit | a4dcff92312b99670e9a19657f881b00a0404e69 (patch) | |
tree | 6c24c55b4edd2ca7e5fa75de131111b64651e21e /chrome/browser/io_thread.cc | |
parent | 608e1a19df52b2c50015ace4fccbea0a3a4173cd (diff) | |
download | chromium_src-a4dcff92312b99670e9a19657f881b00a0404e69.zip chromium_src-a4dcff92312b99670e9a19657f881b00a0404e69.tar.gz chromium_src-a4dcff92312b99670e9a19657f881b00a0404e69.tar.bz2 |
Land Recent QUIC Changes.
Remove loss detection from QuicConfig now that connection options has
replaced it.
Merge internal change: 75952172
Chromium specific changes:
+ Deleted enable_quic_time_based_loss_detection from
NetworkSession params.
+ Deleted enable-quic-time-based-loss-detection and
disable-quic-time-based-loss-detection command line
switches.
https://codereview.chromium.org/591323003/
Do not timeout QUIC connections when settings the timeouts from
InitializeSession(). Protected by FLAG_quic_timeouts_only_from_alarms
Removes FLAG_quic_timeouts_require_activity which was somewhat broken.
Merge internal change: 75927669
https://codereview.chromium.org/605903002/
Factor out the QUIC timeout alarm setting logic from the CheckForTimeout
method into a new SetTimeout method.
- no behavior change, simply moving QUIC timeout alarm code.
Merge internal change: 75915264
https://codereview.chromium.org/593193005/
Add a timestamp field to QUIC's CachedNetworkParams proto message.
Context in b/17357338, follow-up CL will store CachedNetworkParams and
copy into newly created STKs.
Merge internal change: 75897792
https://codereview.chromium.org/604173002/
Call QuicSentPacketManager's OnPacketSent method and make
OnRetransmittedPacket and OnSerializedPacket private.
Merge internal change: 75830237
https://codereview.chromium.org/593193004/
Change the return type of QuicConnection::CheckForTimeout from bool to
void since it is unused.
Merge internal change: 75724127
https://codereview.chromium.org/604163002/
Test-only. Remove calls to OnSerializedPacket from
QuicSentPacketManagerTest, in preparation for OnSerializedPacket to be
removed.
Merge internal change: 75716236
https://codereview.chromium.org/600823006/
R=rch@chromium.org, sky@chromium.org
Added sky@ for OWNERS approval for chrome/browser and chrome/common
changes
Review URL: https://codereview.chromium.org/605733006
Cr-Commit-Position: refs/heads/master@{#297208}
Diffstat (limited to 'chrome/browser/io_thread.cc')
-rw-r--r-- | chrome/browser/io_thread.cc | 27 |
1 files changed, 0 insertions, 27 deletions
diff --git a/chrome/browser/io_thread.cc b/chrome/browser/io_thread.cc index 5ce6ca7..d42723e 100644 --- a/chrome/browser/io_thread.cc +++ b/chrome/browser/io_thread.cc @@ -122,8 +122,6 @@ const char kQuicFieldTrialEnabledGroupName[] = "Enabled"; const char kQuicFieldTrialHttpsEnabledGroupName[] = "HttpsEnabled"; const char kQuicFieldTrialPacketLengthSuffix[] = "BytePackets"; const char kQuicFieldTrialPacingSuffix[] = "WithPacing"; -const char kQuicFieldTrialTimeBasedLossDetectionSuffix[] = - "WithTimeBasedLossDetection"; // The SPDY trial composes two different trial plus control groups: // * A "holdback" group with SPDY disabled, and corresponding control @@ -1015,8 +1013,6 @@ void IOThread::InitializeNetworkSessionParamsFromGlobals( ¶ms->enable_websocket_over_spdy); globals.enable_quic.CopyToIfSet(¶ms->enable_quic); - globals.enable_quic_time_based_loss_detection.CopyToIfSet( - ¶ms->enable_quic_time_based_loss_detection); globals.quic_always_require_handshake_confirmation.CopyToIfSet( ¶ms->quic_always_require_handshake_confirmation); globals.quic_disable_connection_pooling.CopyToIfSet( @@ -1169,9 +1165,6 @@ void IOThread::ConfigureQuicGlobals( bool enable_quic = ShouldEnableQuic(command_line, quic_trial_group); globals->enable_quic.set(enable_quic); if (enable_quic) { - globals->enable_quic_time_based_loss_detection.set( - ShouldEnableQuicTimeBasedLossDetection(command_line, quic_trial_group, - quic_trial_params)); globals->quic_always_require_handshake_confirmation.set( ShouldQuicAlwaysRequireHandshakeConfirmation(quic_trial_params)); globals->quic_disable_connection_pooling.set( @@ -1334,26 +1327,6 @@ double IOThread::GetAlternateProtocolProbabilityThreshold( } // static -bool IOThread::ShouldEnableQuicTimeBasedLossDetection( - const CommandLine& command_line, - base::StringPiece quic_trial_group, - const VariationParameters& quic_trial_params) { - if (command_line.HasSwitch(switches::kEnableQuicTimeBasedLossDetection)) - return true; - - if (command_line.HasSwitch(switches::kDisableQuicTimeBasedLossDetection)) - return false; - - if (LowerCaseEqualsASCII( - GetVariationParam(quic_trial_params, "enable_time_based_loss_detection"), - "true")) - return true; - - return quic_trial_group.ends_with( - kQuicFieldTrialTimeBasedLossDetectionSuffix); -} - -// static bool IOThread::ShouldQuicAlwaysRequireHandshakeConfirmation( const VariationParameters& quic_trial_params) { return LowerCaseEqualsASCII( |