commit 00e7ec69cdfbf3ddc2cd26efd54f46761609198d
author fayang <fayang@google.com> Mon Jul 27 09:26:53 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Jul 27 09:27:31 2020 -0700
tree 9972a68ca9c204968121b86ec791155a9e8a568d
parent af8e330fa6f219b8ce147d893f01f54f6f1ad38c

Deprecate gfe2_reloadable_flag_quic_always_send_earliest_ack.

PiperOrigin-RevId: 323375438
Change-Id: Icd4bbfd06c9e9431afa4b05f1e0454dcaacfb875

quic/core/quic_connection_test.cc

diff --git a/quic/core/quic_connection_test.cc b/quic/core/quic_connection_test.cc
index bf61ade..59f12bb 100644
--- a/quic/core/quic_connection_test.cc
+++ b/quic/core/quic_connection_test.cc
@@ -10945,22 +10945,12 @@
             connection_.GetAckAlarm()->deadline());
 
   // Ack alarm fires early.
-  if (GetQuicReloadableFlag(quic_always_send_earliest_ack)) {
-    // Verify the earliest ACK is flushed.
-    EXPECT_CALL(*send_algorithm_, OnPacketSent(_, _, _, _, _)).Times(1);
-  } else {
-    EXPECT_CALL(*send_algorithm_, OnPacketSent(_, _, _, _, _)).Times(0);
-  }
+  // Verify the earliest ACK is flushed.
+  EXPECT_CALL(*send_algorithm_, OnPacketSent(_, _, _, _, _)).Times(1);
   connection_.GetAckAlarm()->Fire();
   EXPECT_TRUE(connection_.HasPendingAcks());
-  if (GetQuicReloadableFlag(quic_always_send_earliest_ack)) {
-    EXPECT_EQ(clock_.ApproximateNow() + DefaultDelayedAckTime(),
-              connection_.GetAckAlarm()->deadline());
-  } else {
-    // No forward progress has been made.
-    EXPECT_EQ(clock_.ApproximateNow() + kAlarmGranularity,
-              connection_.GetAckAlarm()->deadline());
-  }
+  EXPECT_EQ(clock_.ApproximateNow() + DefaultDelayedAckTime(),
+            connection_.GetAckAlarm()->deadline());
 }
 
 TEST_P(QuicConnectionTest, ClientOnlyBlackholeDetectionClient) {