Deprecate --gfe2_reloadable_flag_quic_crypto_noop_if_disconnected_after_process_chlo.
PiperOrigin-RevId: 431492136
diff --git a/quic/core/quic_crypto_server_stream.cc b/quic/core/quic_crypto_server_stream.cc
index 1f8b026..6bac704 100644
--- a/quic/core/quic_crypto_server_stream.cc
+++ b/quic/core/quic_crypto_server_stream.cc
@@ -168,16 +168,12 @@
AdjustTestValue("quic::QuicCryptoServerStream::after_process_client_hello",
session());
- if (noop_if_disconnected_after_process_chlo_) {
- QUIC_RELOADABLE_FLAG_COUNT(
- quic_crypto_noop_if_disconnected_after_process_chlo);
- if (!session()->connection()->connected()) {
- QUIC_CODE_COUNT(quic_crypto_disconnected_after_process_client_hello);
- QUIC_LOG_FIRST_N(INFO, 10)
- << "After processing CHLO, QUIC connection has been closed with code "
- << session()->error() << ", details: " << session()->error_details();
- return;
- }
+ if (!session()->connection()->connected()) {
+ QUIC_CODE_COUNT(quic_crypto_disconnected_after_process_client_hello);
+ QUIC_LOG_FIRST_N(INFO, 10)
+ << "After processing CHLO, QUIC connection has been closed with code "
+ << session()->error() << ", details: " << session()->error_details();
+ return;
}
const CryptoHandshakeMessage& message = result.client_hello;
diff --git a/quic/core/quic_crypto_server_stream.h b/quic/core/quic_crypto_server_stream.h
index 05f68e6..acb8ae2 100644
--- a/quic/core/quic_crypto_server_stream.h
+++ b/quic/core/quic_crypto_server_stream.h
@@ -256,8 +256,6 @@
bool encryption_established_;
bool one_rtt_keys_available_;
bool one_rtt_packet_decrypted_;
- const bool noop_if_disconnected_after_process_chlo_ = GetQuicReloadableFlag(
- quic_crypto_noop_if_disconnected_after_process_chlo);
QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters>
crypto_negotiated_params_;
};
diff --git a/quic/core/quic_flags_list.h b/quic/core/quic_flags_list.h
index 88f7957..3399f1b 100644
--- a/quic/core/quic_flags_list.h
+++ b/quic/core/quic_flags_list.h
@@ -39,8 +39,6 @@
QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_can_send_ack_frequency, true)
// If true, allow client to enable BBRv2 on server via connection option \'B2ON\'.
QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_allow_client_enabled_bbr_v2, false)
-// If true, change QuicCryptoServerStream::FinishProcessingHandshakeMessageAfterProcessClientHello to noop if connection is disconnected.
-QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_crypto_noop_if_disconnected_after_process_chlo, true)
// If true, close read side but not write side in QuicSpdyStream::OnStreamReset().
QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_fix_on_stream_reset, true)
// If true, default on PTO which unifies TLP + RTO loss recovery.