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);
