gfe-relnote: reset has_non_app_limited_sample in QuicConnectionStats to false once handshake is complete. Logging only change, not protected.

This is only a temporary fix that discounts handshake samples as non app-limited samples in the stats.

PiperOrigin-RevId: 301227272
Change-Id: Ie1b9adaa42f170a3b130805e248fd923bc4b2b3a

quic/core/quic_connection.cc

diff --git a/quic/core/quic_connection.cc b/quic/core/quic_connection.cc
index 566d477..c067f7a 100644
--- a/quic/core/quic_connection.cc
+++ b/quic/core/quic_connection.cc
@@ -1690,6 +1690,10 @@
   return stats_;
 }
 
+void QuicConnection::ResetHasNonAppLimitedSampleAfterHandshakeCompletion() {
+  stats_.has_non_app_limited_sample = false;
+}
+
 void QuicConnection::OnCoalescedPacket(const QuicEncryptedPacket& packet) {
   QueueCoalescedPacket(packet);
 }

quic/core/quic_connection.h

diff --git a/quic/core/quic_connection.h b/quic/core/quic_connection.h
index e0870d4..8cb1d37 100644
--- a/quic/core/quic_connection.h
+++ b/quic/core/quic_connection.h
@@ -432,6 +432,11 @@
   // Returns statistics tracked for this connection.
   const QuicConnectionStats& GetStats();
 
+  // Mark stats_.has_non_app_limited_sample as false.
+  // TODO(b/151166631) Remove this once the proper fix in b/151166631 is rolled
+  // out.
+  void ResetHasNonAppLimitedSampleAfterHandshakeCompletion();
+
   // Processes an incoming UDP packet (consisting of a QuicEncryptedPacket) from
   // the peer.
   // In a client, the packet may be "stray" and have a different connection ID

quic/core/quic_session.cc

diff --git a/quic/core/quic_session.cc b/quic/core/quic_session.cc
index 44561e1..83792db 100644
--- a/quic/core/quic_session.cc
+++ b/quic/core/quic_session.cc
@@ -1376,6 +1376,7 @@
         // to the client.
         control_frame_manager_.WriteOrBufferHandshakeDone();
       }
+      connection_->ResetHasNonAppLimitedSampleAfterHandshakeCompletion();
       break;
     default:
       QUIC_BUG << "Unknown encryption level: "