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

net/quic/chromium/mock_crypto_client_stream_factory.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/chromium/mock_crypto_client_stream_factory.h"
 
 #include "base/lazy_instance.h"
 #include "net/quic/chromium/quic_chromium_client_session.h"
 #include "net/quic/core/quic_crypto_client_stream.h"
 
@@ skipping 16 matching lines @@
 MockCryptoClientStreamFactory::CreateQuicCryptoClientStream(
     const QuicServerId& server_id,
     QuicChromiumClientSession* session,
     std::unique_ptr<ProofVerifyContext> /*proof_verify_context*/,
     QuicCryptoClientConfig* crypto_config) {
   const ProofVerifyDetailsChromium* proof_verify_details = nullptr;
   if (!proof_verify_details_queue_.empty()) {
     proof_verify_details = proof_verify_details_queue_.front();
     proof_verify_details_queue_.pop();
   }
+
+  if (handshake_mode_ == MockCryptoClientStream::USE_DEFAULT_CRYPTO_STREAM) {
+    return new QuicCryptoClientStream(server_id, session, nullptr,
+                                      crypto_config, session);
+  }

[Ryan Hamilton, 2017/03/22 14:13:01]

nit: Sorry, I should have noticed this last time. Since the real crypto stream
does not use the proof verify details, lets move this block up to be the first
line in the method.

[Zhongyi Shi, 2017/03/22 18:53:10]

Done.

+
   last_stream_ = new MockCryptoClientStream(
       server_id, session, nullptr, *(config_.get()), crypto_config,
       handshake_mode_, proof_verify_details);
   return last_stream_;
 }
 
 }  // namespace net