Deprecate --gfe2_reloadable_flag_quic_send_goaway_with_connection_close.

PiperOrigin-RevId: 359928956
Change-Id: Ib736421786e69695fd172d30f195a5ffbde7df52

quic/core/http/quic_spdy_session.cc

diff --git a/quic/core/http/quic_spdy_session.cc b/quic/core/http/quic_spdy_session.cc
index b6750fe..e8a473f 100644
--- a/quic/core/http/quic_spdy_session.cc
+++ b/quic/core/http/quic_spdy_session.cc
@@ -1483,15 +1483,12 @@
 }
 
 void QuicSpdySession::BeforeConnectionCloseSent() {
-  if (!GetQuicReloadableFlag(quic_send_goaway_with_connection_close) ||
-      !VersionUsesHttp3(transport_version()) || !IsEncryptionEstablished()) {
+  if (!VersionUsesHttp3(transport_version()) || !IsEncryptionEstablished()) {
     return;
   }
 
   QUICHE_DCHECK_EQ(perspective(), Perspective::IS_SERVER);
 
-  QUIC_CODE_COUNT(quic_send_goaway_with_connection_close);
-
   QuicStreamId stream_id =
       GetLargestPeerCreatedStreamId(/*unidirectional = */ false);
 

quic/core/http/quic_spdy_session_test.cc

diff --git a/quic/core/http/quic_spdy_session_test.cc b/quic/core/http/quic_spdy_session_test.cc
index d231f78..2cf98ab 100644
--- a/quic/core/http/quic_spdy_session_test.cc
+++ b/quic/core/http/quic_spdy_session_test.cc
@@ -3112,13 +3112,11 @@
   EXPECT_EQ(stream_id, QuicSessionPeer::GetLargestPeerCreatedStreamId(
                            &session_, /*unidirectional = */ false));
 
-  if (GetQuicReloadableFlag(quic_send_goaway_with_connection_close)) {
-    // Stream with stream_id is already received and potentially processed,
-    // therefore a GOAWAY frame is sent with the next stream ID.
-    EXPECT_CALL(debug_visitor,
-                OnGoAwayFrameSent(
-                    stream_id + QuicUtils::StreamIdDelta(transport_version())));
-  }
+  // Stream with stream_id is already received and potentially processed,
+  // therefore a GOAWAY frame is sent with the next stream ID.
+  EXPECT_CALL(debug_visitor,
+              OnGoAwayFrameSent(stream_id +
+                                QuicUtils::StreamIdDelta(transport_version())));
 
   // Close connection.
   EXPECT_CALL(*writer_, WritePacket(_, _, _, _, _))

quic/core/quic_flags_list.h

diff --git a/quic/core/quic_flags_list.h b/quic/core/quic_flags_list.h
index d2a66e0..7750948 100644
--- a/quic/core/quic_flags_list.h
+++ b/quic/core/quic_flags_list.h
@@ -48,7 +48,6 @@
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_h3_datagram, false)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_pass_path_response_to_validator, false)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_require_handshake_confirmation, false)
-QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_send_goaway_with_connection_close, true)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_send_path_response, false)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_send_timestamps, false)
 QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_send_tls_crypto_error_code, true)