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.