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

net/quic/chromium/quic_chromium_client_session.cc

Index: net/quic/chromium/quic_chromium_client_session.cc
diff --git a/net/quic/chromium/quic_chromium_client_session.cc b/net/quic/chromium/quic_chromium_client_session.cc
index d783551a80145c58032a9ce31a09b2446bc0f9d6..115faac314756485c856c235b891ffc6b65931ef 100644
--- a/net/quic/chromium/quic_chromium_client_session.cc
+++ b/net/quic/chromium/quic_chromium_client_session.cc
@@ -670,6 +670,9 @@ int QuicChromiumClientSession::CryptoConnect(
   DCHECK(flow_controller());
   crypto_stream_->CryptoConnect();
 
+  if (!connection()->connected())

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

nit: Maybe add a comment here explaining why we need to check handshake state
after CryptoConnect().

Maybe a TODO here with a bug number to investigate whether
crypto_stream_->CryptoConnect() could take a callback as a parameter? It will
make some of the code easier to write and less error-prone. 

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

Done.

+    return ERR_QUIC_HANDSHAKE_FAILED;
+
   if (IsCryptoHandshakeConfirmed()) {
     connect_timing_.connect_end = base::TimeTicks::Now();
     return OK;