QUIC - Construct crypto_stream_ in QuicClientSession::InitializeSession
instread of QuicClientSession's constructor to make it easier for server
side code to do the same.
Call QuicClientSessionBase::InitializeSession() after constructing
QuicCryptoClientStream.
This is done per discussion in interal change: 72671781
The following are the earlier related issues:
https://codereview.chromium.org/393953009/
https://codereview.chromium.org/407193002/
[email protected]
Review URL: https://codereview.chromium.org/481103003
Cr-Commit-Position: refs/heads/master@{#291812}
diff --git a/net/quic/quic_http_stream_test.cc b/net/quic/quic_http_stream_test.cc
index 7ebf8e5..14b1a11 100644
--- a/net/quic/quic_http_stream_test.cc
+++ b/net/quic/quic_http_stream_test.cc
@@ -228,16 +228,16 @@
new QuicClientSession(connection_,
scoped_ptr<DatagramClientSocket>(socket),
NULL,
- &crypto_client_stream_factory_,
&transport_security_state_,
make_scoped_ptr((QuicServerInfo*)NULL),
- QuicServerId(kServerHostname, kServerPort,
- false, PRIVACY_MODE_DISABLED),
- DefaultQuicConfig(), &crypto_config_,
+ DefaultQuicConfig(),
base::MessageLoop::current()->
message_loop_proxy().get(),
NULL));
- session_->InitializeSession();
+ session_->InitializeSession(QuicServerId(kServerHostname, kServerPort,
+ false, PRIVACY_MODE_DISABLED),
+ &crypto_config_,
+ &crypto_client_stream_factory_);
session_->GetCryptoStream()->CryptoConnect();
EXPECT_TRUE(session_->IsCryptoHandshakeConfirmed());
stream_.reset(use_closing_stream_ ?
