commit 631f160a241fde1817b74dacf327d892d7ae86a8
author dschinazi <dschinazi@google.com> Tue May 19 10:10:22 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Tue May 19 10:11:01 2020 -0700
tree feb1042a1d2654160801b61360891a201d88d0d3
parent 9d0a88542030288b1e02646485b929abb1f0f481

Add transport parameters to QuicConnectionDebugVisitor

This will allow us to display transport parameters in netlog. I've confirmed locally that this behaves as expected in Chrome and successfully displays the transport parameters.

Changes to quicconnectiondebugvisitor, not used in prod

PiperOrigin-RevId: 312302229
Change-Id: Ifa4aedd0db640d10cfc45504208e1d755fe16f30

quic/core/tls_client_handshaker.cc

diff --git a/quic/core/tls_client_handshaker.cc b/quic/core/tls_client_handshaker.cc
index 30a9ac4..aed628e 100644
--- a/quic/core/tls_client_handshaker.cc
+++ b/quic/core/tls_client_handshaker.cc
@@ -216,6 +216,9 @@
     params.google_quic_params->SetStringPiece(kUAID, user_agent_id_);
   }
 
+  // Notify QuicConnectionDebugVisitor.
+  session()->connection()->OnTransportParametersSent(params);
+
   std::vector<uint8_t> param_bytes;
   return SerializeTransportParameters(session()->connection()->version(),
                                       params, &param_bytes) &&
@@ -244,6 +247,10 @@
     return false;
   }
 
+  // Notify QuicConnectionDebugVisitor.
+  session()->connection()->OnTransportParametersReceived(
+      *received_transport_params_);
+
   // When interoperating with non-Google implementations that do not send
   // the version extension, set it to what we expect.
   if (received_transport_params_->version == 0) {