Deprecate --quic_dispatcher_send_connection_close_for_tls_alerts.

PiperOrigin-RevId: 475350434

quiche/quic/core/quic_dispatcher.cc

diff --git a/quiche/quic/core/quic_dispatcher.cc b/quiche/quic/core/quic_dispatcher.cc
index 1ea2783..74ba7d1 100644
--- a/quiche/quic/core/quic_dispatcher.cc
+++ b/quiche/quic/core/quic_dispatcher.cc
@@ -361,10 +361,6 @@
       << "Trying to create dispatcher without any supported versions";
   QUIC_DLOG(INFO) << "Created QuicDispatcher with versions: "
                   << ParsedQuicVersionVectorToString(GetSupportedVersions());
-  if (send_connection_close_for_tls_alerts_) {
-    QUIC_RESTART_FLAG_COUNT_N(
-        quic_dispatcher_send_connection_close_for_tls_alerts, 1, 3);
-  }
 }
 
 QuicDispatcher::~QuicDispatcher() {
@@ -752,10 +748,7 @@
         TryExtractChloOrBufferEarlyPacket(*packet_info);
     auto& parsed_chlo = extract_chlo_result.parsed_chlo;
 
-    if (send_connection_close_for_tls_alerts_ &&
-        extract_chlo_result.tls_alert.has_value()) {
-      QUIC_RESTART_FLAG_COUNT_N(
-          quic_dispatcher_send_connection_close_for_tls_alerts, 2, 3);
+    if (extract_chlo_result.tls_alert.has_value()) {
       QUIC_BUG_IF(quic_dispatcher_parsed_chlo_and_tls_alert_coexist_1,
                   parsed_chlo.has_value())
           << "parsed_chlo and tls_alert should not be set at the same time.";
@@ -867,9 +860,7 @@
       }
     }
 
-    if (send_connection_close_for_tls_alerts_ && result.tls_alert.has_value()) {
-      QUIC_RESTART_FLAG_COUNT_N(
-          quic_dispatcher_send_connection_close_for_tls_alerts, 3, 3);
+    if (result.tls_alert.has_value()) {
       QUIC_BUG_IF(quic_dispatcher_parsed_chlo_and_tls_alert_coexist_2,
                   has_full_tls_chlo)
           << "parsed_chlo and tls_alert should not be set at the same time.";

quiche/quic/core/quic_dispatcher.h

diff --git a/quiche/quic/core/quic_dispatcher.h b/quiche/quic/core/quic_dispatcher.h
index e2b7761..9cfc3b0 100644
--- a/quiche/quic/core/quic_dispatcher.h
+++ b/quiche/quic/core/quic_dispatcher.h
@@ -479,9 +479,6 @@
   // destination connection ID length of all IETF long headers.
   bool should_update_expected_server_connection_id_length_;
 
-  const bool send_connection_close_for_tls_alerts_ =
-      GetQuicRestartFlag(quic_dispatcher_send_connection_close_for_tls_alerts);
-
   ConnectionIdGeneratorInterface& connection_id_generator_;
 };
 

quiche/quic/core/quic_flags_list.h

diff --git a/quiche/quic/core/quic_flags_list.h b/quiche/quic/core/quic_flags_list.h
index 3cb93c4..4ba71d6 100644
--- a/quiche/quic/core/quic_flags_list.h
+++ b/quiche/quic/core/quic_flags_list.h
@@ -59,8 +59,6 @@
 QUIC_FLAG(quic_reloadable_flag_quic_flush_pending_frames_and_padding_bytes_on_migration, true)
 // If true, ietf connection migration is no longer conditioned on connection option RVCM.
 QUIC_FLAG(quic_reloadable_flag_quic_remove_connection_migration_connection_option_v2, false)
-// If true, if a fatal tls alert is raised while extracting CHLO, QuicDispatcher will send a connection close.
-QUIC_FLAG(quic_restart_flag_quic_dispatcher_send_connection_close_for_tls_alerts, true)
 // If true, include stream information in idle timeout connection close detail.
 QUIC_FLAG(quic_reloadable_flag_quic_add_stream_info_to_idle_close_detail, true)
 // If true, quic server will send ENABLE_CONNECT_PROTOCOL setting and and endpoint will validate required request/response headers and extended CONNECT mechanism and update code counts of valid/invalid headers.