Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(122)

Unified Diff: net/quic/quic_client_session.cc

Issue 407193002: QUIC - undo the chromium specific changes from the following CL which (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/quic/quic_client_session.h ('k') | net/quic/quic_client_session_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_client_session.cc
diff --git a/net/quic/quic_client_session.cc b/net/quic/quic_client_session.cc
index fa6a21156cb8c4c93b9e6937f238dbd5617e8b8a..067009e2e5928a7767f9f0836435a502e1d3bc15 100644
--- a/net/quic/quic_client_session.cc
+++ b/net/quic/quic_client_session.cc
@@ -132,15 +132,20 @@ void QuicClientSession::StreamRequest::OnRequestCompleteFailure(int rv) {
ResetAndReturn(&callback_).Run(rv);
}
-QuicClientSession::QuicClientSession(QuicConnection* connection,
- scoped_ptr<DatagramClientSocket> socket,
- scoped_ptr<QuicDefaultPacketWriter> writer,
- QuicStreamFactory* stream_factory,
- scoped_ptr<QuicServerInfo> server_info,
- const QuicConfig& config,
- base::TaskRunner* task_runner,
- NetLog* net_log)
+QuicClientSession::QuicClientSession(
+ QuicConnection* connection,
+ scoped_ptr<DatagramClientSocket> socket,
+ scoped_ptr<QuicDefaultPacketWriter> writer,
+ QuicStreamFactory* stream_factory,
+ QuicCryptoClientStreamFactory* crypto_client_stream_factory,
+ scoped_ptr<QuicServerInfo> server_info,
+ const QuicServerId& server_id,
+ const QuicConfig& config,
+ QuicCryptoClientConfig* crypto_config,
+ base::TaskRunner* task_runner,
+ NetLog* net_log)
: QuicClientSessionBase(connection, config),
+ server_host_port_(server_id.host_port_pair()),
require_confirmation_(false),
stream_factory_(stream_factory),
socket_(socket.Pass()),
@@ -155,15 +160,6 @@ QuicClientSession::QuicClientSession(QuicConnection* connection,
num_packets_read_(0),
going_away_(false),
weak_factory_(this) {
- connection->set_debug_visitor(&logger_);
-}
-
-void QuicClientSession::InitializeSession(
- const QuicServerId& server_id,
- QuicCryptoClientConfig* crypto_config,
- QuicCryptoClientStreamFactory* crypto_client_stream_factory) {
- QuicClientSessionBase::InitializeSession();
- server_host_port_.reset(new HostPortPair(server_id.host_port_pair()));
crypto_stream_.reset(
crypto_client_stream_factory ?
crypto_client_stream_factory->CreateQuicCryptoClientStream(
@@ -172,6 +168,7 @@ void QuicClientSession::InitializeSession(
new ProofVerifyContextChromium(net_log_),
crypto_config));
+ connection->set_debug_visitor(&logger_);
// TODO(rch): pass in full host port proxy pair
net_log_.BeginEvent(
NetLog::TYPE_QUIC_SESSION,
@@ -503,7 +500,7 @@ bool QuicClientSession::CanPool(const std::string& hostname) const {
if (ssl_info.channel_id_sent &&
ServerBoundCertService::GetDomainForHost(hostname) !=
- ServerBoundCertService::GetDomainForHost(server_host_port_->host())) {
+ ServerBoundCertService::GetDomainForHost(server_host_port_.host())) {
return false;
}
« no previous file with comments | « net/quic/quic_client_session.h ('k') | net/quic/quic_client_session_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698