Disable sending min_ack_delay in transport parameters on the client side. It is a no-op for google service and not flag protected.

Apparently, non-Google servers might react to this parameter and therefore it should be sent by default until it is fully tested.

PiperOrigin-RevId: 325226780
Change-Id: Id35c9862a1afcf8c94dce5ec8c2bf4617977890f

quic/core/quic_session.cc

diff --git a/quic/core/quic_session.cc b/quic/core/quic_session.cc
index 4d6ea0e..b2ac3ab 100644
--- a/quic/core/quic_session.cc
+++ b/quic/core/quic_session.cc
@@ -127,7 +127,6 @@
   connection_->SetDataProducer(this);
   connection_->SetFromConfig(config_);
   if (perspective() == Perspective::IS_CLIENT && version().UsesTls()) {
-    config_.SetMinAckDelayMs(kDefaultMinAckDelayTimeMs);
     if (GetQuicReloadableFlag(quic_support_handshake_done_in_t050) &&
         !version().HasHandshakeDone()) {
       config_.SetSupportHandshakeDone();

quic/core/quic_session_test.cc

diff --git a/quic/core/quic_session_test.cc b/quic/core/quic_session_test.cc
index 1a5a290..3973bd5 100644
--- a/quic/core/quic_session_test.cc
+++ b/quic/core/quic_session_test.cc
@@ -2208,14 +2208,6 @@
   session_.OnStreamFrame(frame);
 }
 
-TEST_P(QuicSessionTestClient, MinAckDelaySetOnTheClientQuicConfig) {
-  if (!VersionUsesHttp3(transport_version())) {
-    return;
-  }
-  ASSERT_EQ(session_.config()->GetMinAckDelayToSendMs(),
-            kDefaultMinAckDelayTimeMs);
-}
-
 TEST_P(QuicSessionTestClient, FailedToCreateStreamIfTooCloseToIdleTimeout) {
   connection_->SetDefaultEncryptionLevel(ENCRYPTION_FORWARD_SECURE);
   EXPECT_TRUE(session_.CanOpenNextOutgoingBidirectionalStream());