Deprecate gfe2_reloadable_flag_quic_check_keys_before_writing.

PiperOrigin-RevId: 343912495
Change-Id: Ifb29e259d22e1a1a81bd14fc6b3911afdb19fab7

quic/core/quic_connection.cc

diff --git a/quic/core/quic_connection.cc b/quic/core/quic_connection.cc
index bca806e..19d6ac3 100644
--- a/quic/core/quic_connection.cc
+++ b/quic/core/quic_connection.cc
@@ -4906,14 +4906,11 @@
     return;
   }
 
-  if (check_keys_before_writing_) {
-    QUIC_RELOADABLE_FLAG_COUNT_N(quic_check_keys_before_writing, 1, 2);
-    if (!framer_.HasAnEncrypterForSpace(space)) {
-      QUIC_BUG << ENDPOINT
-               << "Try to bundle crypto with ACK with missing key of space "
-               << PacketNumberSpaceToString(space);
-      return;
-    }
+  if (!framer_.HasAnEncrypterForSpace(space)) {
+    QUIC_BUG << ENDPOINT
+             << "Try to bundle crypto with ACK with missing key of space "
+             << PacketNumberSpaceToString(space);
+    return;
   }
 
   sent_packet_manager_.RetransmitDataOfSpaceIfAny(space);
@@ -4939,12 +4936,9 @@
     if (!ack_timeout.IsInitialized()) {
       continue;
     }
-    if (check_keys_before_writing_) {
-      QUIC_RELOADABLE_FLAG_COUNT_N(quic_check_keys_before_writing, 2, 2);
-      if (!framer_.HasAnEncrypterForSpace(static_cast<PacketNumberSpace>(i))) {
-        // The key has been dropped.
-        continue;
-      }
+    if (!framer_.HasAnEncrypterForSpace(static_cast<PacketNumberSpace>(i))) {
+      // The key has been dropped.
+      continue;
     }
     if (ack_timeout > clock_->ApproximateNow() &&
         ack_timeout > earliest_ack_timeout) {

quic/core/quic_connection.h

diff --git a/quic/core/quic_connection.h b/quic/core/quic_connection.h
index 599331d..823fa93 100644
--- a/quic/core/quic_connection.h
+++ b/quic/core/quic_connection.h
@@ -1140,8 +1140,6 @@
     can_receive_ack_frequency_frame_ = true;
   }
 
-  bool check_keys_before_writing() const { return check_keys_before_writing_; }
-
   bool is_processing_packet() const { return framer_.is_processing_packet(); }
 
   bool encrypted_control_frames() const { return encrypted_control_frames_; }
@@ -1959,9 +1957,6 @@
   const bool fix_out_of_order_sending_ =
       GetQuicReloadableFlag(quic_fix_out_of_order_sending2);
 
-  const bool check_keys_before_writing_ =
-      GetQuicReloadableFlag(quic_check_keys_before_writing);
-
   const bool encrypted_control_frames_;
 
   const bool use_encryption_level_context_;

quic/core/quic_flags_list.h

diff --git a/quic/core/quic_flags_list.h b/quic/core/quic_flags_list.h
index b021175..c5d0a9f 100644
--- a/quic/core/quic_flags_list.h
+++ b/quic/core/quic_flags_list.h
@@ -16,7 +16,6 @@
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_bbr2_startup_loss_exit_use_max_delivered, true)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_bbr2_support_max_bootstrap_cwnd, true)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_can_send_ack_frequency, true)
-QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_check_keys_before_writing, true)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_close_connection_in_on_can_write_with_blocked_writer, true)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_connection_set_initial_self_address, true)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_conservative_bursts, false)

quic/core/quic_session.cc

diff --git a/quic/core/quic_session.cc b/quic/core/quic_session.cc
index 44fe0c8..2e1895e 100644
--- a/quic/core/quic_session.cc
+++ b/quic/core/quic_session.cc
@@ -795,8 +795,7 @@
                                    QuicStreamOffset offset,
                                    TransmissionType type) {
   DCHECK(QuicVersionUsesCryptoFrames(transport_version()));
-  if (connection()->check_keys_before_writing() &&
-      !connection()->framer().HasEncrypterOfEncryptionLevel(level)) {
+  if (!connection()->framer().HasEncrypterOfEncryptionLevel(level)) {
     const std::string error_details = quiche::QuicheStrCat(
         "Try to send crypto data with missing keys of encryption level: ",
         EncryptionLevelToString(level));