gfe-relnote: (n/a) In QUIC BBRv1, do not exit startup on loss when LRTT is set. Not protected because LRTT is not used except for 1% of QuicCanary, which I'll disable shortly. PiperOrigin-RevId: 289005413 Change-Id: Ib96af95526d811f867d970af09e881e2037f218b
diff --git a/quic/core/congestion_control/bbr_sender.cc b/quic/core/congestion_control/bbr_sender.cc index bb63b55..f8d22db 100644 --- a/quic/core/congestion_control/bbr_sender.cc +++ b/quic/core/congestion_control/bbr_sender.cc
@@ -709,8 +709,7 @@ } rounds_without_bandwidth_gain_++; - if ((rounds_without_bandwidth_gain_ >= num_startup_rtts_) || - (exit_startup_on_loss_ && InRecovery())) { + if (rounds_without_bandwidth_gain_ >= num_startup_rtts_) { DCHECK(has_non_app_limited_sample_); is_at_full_bandwidth_ = true; }
diff --git a/quic/core/congestion_control/bbr_sender_test.cc b/quic/core/congestion_control/bbr_sender_test.cc index c8b1b77..ddb1077 100644 --- a/quic/core/congestion_control/bbr_sender_test.cc +++ b/quic/core/congestion_control/bbr_sender_test.cc
@@ -941,7 +941,7 @@ } // Test exiting STARTUP earlier upon loss due to the LRTT connection option. -TEST_F(BbrSenderTest, SimpleTransferLRTTStartup) { +TEST_F(BbrSenderTest, DISABLED_SimpleTransferLRTTStartup) { CreateDefaultSetup(); SetConnectionOption(kLRTT); @@ -969,7 +969,7 @@ } // Test exiting STARTUP earlier upon loss due to the LRTT connection option. -TEST_F(BbrSenderTest, SimpleTransferLRTTStartupSmallBuffer) { +TEST_F(BbrSenderTest, DISABLED_SimpleTransferLRTTStartupSmallBuffer) { CreateSmallBufferSetup(); SetConnectionOption(kLRTT);