gfe-relnote: Deprecate gfe2_reloadable_flag_quic_enable_ietf_loss_detection. PiperOrigin-RevId: 289430458 Change-Id: I16ff18596ddbcf6485be30b501105bb36f95aa35
diff --git a/quic/core/quic_sent_packet_manager.cc b/quic/core/quic_sent_packet_manager.cc index bcd7590..72ca436 100644 --- a/quic/core/quic_sent_packet_manager.cc +++ b/quic/core/quic_sent_packet_manager.cc
@@ -266,44 +266,37 @@ uber_loss_algorithm_.SetLossDetectionType(kLazyFack); } } - if (GetQuicReloadableFlag(quic_enable_ietf_loss_detection)) { - if (config.HasClientRequestedIndependentOption(kILD0, perspective)) { - QUIC_RELOADABLE_FLAG_COUNT_N(quic_enable_ietf_loss_detection, 1, 5); - uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); - if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { - uber_loss_algorithm_.SetReorderingShift(kDefaultIetfLossDelayShift); - uber_loss_algorithm_.DisableAdaptiveReorderingThreshold(); - } + if (config.HasClientRequestedIndependentOption(kILD0, perspective)) { + uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); + if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { + uber_loss_algorithm_.SetReorderingShift(kDefaultIetfLossDelayShift); + uber_loss_algorithm_.DisableAdaptiveReorderingThreshold(); } - if (config.HasClientRequestedIndependentOption(kILD1, perspective)) { - QUIC_RELOADABLE_FLAG_COUNT_N(quic_enable_ietf_loss_detection, 2, 5); - uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); - uber_loss_algorithm_.SetReorderingShift(kDefaultLossDelayShift); - if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { - uber_loss_algorithm_.DisableAdaptiveReorderingThreshold(); - } + } + if (config.HasClientRequestedIndependentOption(kILD1, perspective)) { + uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); + uber_loss_algorithm_.SetReorderingShift(kDefaultLossDelayShift); + if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { + uber_loss_algorithm_.DisableAdaptiveReorderingThreshold(); } - if (config.HasClientRequestedIndependentOption(kILD2, perspective)) { - QUIC_RELOADABLE_FLAG_COUNT_N(quic_enable_ietf_loss_detection, 3, 5); - uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); - uber_loss_algorithm_.EnableAdaptiveReorderingThreshold(); - if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { - uber_loss_algorithm_.SetReorderingShift(kDefaultIetfLossDelayShift); - } + } + if (config.HasClientRequestedIndependentOption(kILD2, perspective)) { + uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); + uber_loss_algorithm_.EnableAdaptiveReorderingThreshold(); + if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { + uber_loss_algorithm_.SetReorderingShift(kDefaultIetfLossDelayShift); } - if (config.HasClientRequestedIndependentOption(kILD3, perspective)) { - QUIC_RELOADABLE_FLAG_COUNT_N(quic_enable_ietf_loss_detection, 4, 5); - uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); - uber_loss_algorithm_.SetReorderingShift(kDefaultLossDelayShift); - uber_loss_algorithm_.EnableAdaptiveReorderingThreshold(); - } - if (config.HasClientRequestedIndependentOption(kILD4, perspective)) { - QUIC_RELOADABLE_FLAG_COUNT_N(quic_enable_ietf_loss_detection, 5, 5); - uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); - uber_loss_algorithm_.SetReorderingShift(kDefaultLossDelayShift); - uber_loss_algorithm_.EnableAdaptiveReorderingThreshold(); - uber_loss_algorithm_.EnableAdaptiveTimeThreshold(); - } + } + if (config.HasClientRequestedIndependentOption(kILD3, perspective)) { + uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); + uber_loss_algorithm_.SetReorderingShift(kDefaultLossDelayShift); + uber_loss_algorithm_.EnableAdaptiveReorderingThreshold(); + } + if (config.HasClientRequestedIndependentOption(kILD4, perspective)) { + uber_loss_algorithm_.SetLossDetectionType(kIetfLossDetection); + uber_loss_algorithm_.SetReorderingShift(kDefaultLossDelayShift); + uber_loss_algorithm_.EnableAdaptiveReorderingThreshold(); + uber_loss_algorithm_.EnableAdaptiveTimeThreshold(); } if (config.HasClientSentConnectionOption(kCONH, perspective)) { conservative_handshake_retransmits_ = true;
diff --git a/quic/core/quic_sent_packet_manager_test.cc b/quic/core/quic_sent_packet_manager_test.cc index 86ee40a..a36d86a 100644 --- a/quic/core/quic_sent_packet_manager_test.cc +++ b/quic/core/quic_sent_packet_manager_test.cc
@@ -1799,7 +1799,6 @@ } TEST_F(QuicSentPacketManagerTest, NegotiateIetfLossDetectionFromOptions) { - SetQuicReloadableFlag(quic_enable_ietf_loss_detection, true); if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { EXPECT_EQ(kIetfLossDetection, QuicSentPacketManagerPeer::GetLossAlgorithm(&manager_) @@ -1833,7 +1832,6 @@ TEST_F(QuicSentPacketManagerTest, NegotiateIetfLossDetectionOneFourthRttFromOptions) { - SetQuicReloadableFlag(quic_enable_ietf_loss_detection, true); if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { EXPECT_EQ(kIetfLossDetection, QuicSentPacketManagerPeer::GetLossAlgorithm(&manager_) @@ -1868,7 +1866,6 @@ TEST_F(QuicSentPacketManagerTest, NegotiateIetfLossDetectionAdaptiveReorderingThreshold) { - SetQuicReloadableFlag(quic_enable_ietf_loss_detection, true); if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { EXPECT_EQ(kIetfLossDetection, QuicSentPacketManagerPeer::GetLossAlgorithm(&manager_) @@ -1904,7 +1901,6 @@ TEST_F(QuicSentPacketManagerTest, NegotiateIetfLossDetectionAdaptiveReorderingThreshold2) { - SetQuicReloadableFlag(quic_enable_ietf_loss_detection, true); if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { EXPECT_EQ(kIetfLossDetection, QuicSentPacketManagerPeer::GetLossAlgorithm(&manager_) @@ -1940,7 +1936,6 @@ TEST_F(QuicSentPacketManagerTest, NegotiateIetfLossDetectionAdaptiveReorderingAndTimeThreshold) { - SetQuicReloadableFlag(quic_enable_ietf_loss_detection, true); if (GetQuicRestartFlag(quic_default_on_ietf_loss_detection)) { EXPECT_EQ(kIetfLossDetection, QuicSentPacketManagerPeer::GetLossAlgorithm(&manager_)