QUIC: mark QUIC handshake failed if connection is closed after CryptoConnect

net/quic/test_tools/mock_crypto_client_stream.h

Index: net/quic/test_tools/mock_crypto_client_stream.h
diff --git a/net/quic/test_tools/mock_crypto_client_stream.h b/net/quic/test_tools/mock_crypto_client_stream.h
index 7c8b9fb109998f7f0f3474b0730883c76819e132..c89df1dc86ad992dea89ec2229225caf7429ff7c 100644
--- a/net/quic/test_tools/mock_crypto_client_stream.h
+++ b/net/quic/test_tools/mock_crypto_client_stream.h
@@ -35,6 +35,11 @@ class MockCryptoClientStream : public QuicCryptoClientStream {
     // COLD_START indicates that CryptoConnect will neither establish encryption
     // nor confirm the handshake
     COLD_START,
+
+    // USE_DEFAULT_CRYPTO_STREAM indicates that MockCryptoClientStreamFactory

[xunjieli, 2017/03/23 16:18:02]

Hmm.. even though this is test code, there is layering violation here. Can you
move the enum to the Factory class?

[Zhongyi Shi, 2017/03/23 17:28:50]

I prefer to move this to another CL, since if we move the enum to the factory in
this CL, it will introduce a lot of test changes. Considering this CL will be
merged to other milestones, shall we keep it simple as is here? 

[xunjieli, 2017/03/23 17:43:37]

SG. Can we add a crbug to track it?

+    // will create a QuicCryptoClientStream instead of a
+    // MockCryptoClientStream.
+    USE_DEFAULT_CRYPTO_STREAM,
   };
 
   MockCryptoClientStream(