commit bcc066a4749bc9c770ece2dbee28adcd84e8f088
author renjietang <renjietang@google.com> Tue Apr 21 18:05:57 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Apr 21 18:06:37 2020 -0700
tree 65fa971d3da67601f548d5ce7e6e25e9eb084384
parent 18f1067b7a5dcbd553521f2dbb6eddcb7a7337ce

In TlsClientHandshaker, only insert data into cache if all required states are present.

ApplicationState is now  optional to TlsClientHandshaker because not all applications have it.

gfe-relnote: unused code. not protected.
PiperOrigin-RevId: 307719344
Change-Id: Ib4e359c6a13c9b7b805dca7a1cfd3b1bf89af6ef

quic/test_tools/quic_test_utils.cc

diff --git a/quic/test_tools/quic_test_utils.cc b/quic/test_tools/quic_test_utils.cc
index bb45ae3..9fab452 100644
--- a/quic/test_tools/quic_test_utils.cc
+++ b/quic/test_tools/quic_test_utils.cc
@@ -746,9 +746,11 @@
                                 &push_promise_index_,
                                 config,
                                 supported_versions) {
+  // TODO(b/153726130): Consider adding OnApplicationState calls in tests and
+  // set |has_application_state| to true.
   crypto_stream_ = std::make_unique<QuicCryptoClientStream>(
       server_id, this, crypto_test_utils::ProofVerifyContextForTesting(),
-      crypto_config, this);
+      crypto_config, this, /*has_application_state = */ false);
   Initialize();
 }