QUIC - use QuicSessionKey tuple (host, port, is_https) instead of server_hostname

net/tools/quic/quic_client_session.cc

Index: net/tools/quic/quic_client_session.cc
diff --git a/net/tools/quic/quic_client_session.cc b/net/tools/quic/quic_client_session.cc
index a2d69498346732272cf741c81537bc92bd676360..ab492a30c8cbc567915d465675d553b285c1ddb4 100644
--- a/net/tools/quic/quic_client_session.cc
+++ b/net/tools/quic/quic_client_session.cc
@@ -19,7 +19,8 @@ QuicClientSession::QuicClientSession(
     QuicConnection* connection,
     QuicCryptoClientConfig* crypto_config)
     : QuicSession(connection, config),
-      crypto_stream_(server_hostname, this, crypto_config) {
+      crypto_stream_(server_hostname, connection->peer_address().port(), this,

[wtc, 2014/03/11 01:41:39]

IMPORTANT: it is strange that we get the hostname from a stand-alone argument,
but get the port from the |connection| argument.

[Ryan Hamilton, 2014/03/11 15:47:43]

It looks like QuicStreamFactory has added port to the QuicClientSession
constructor, but here in QuicClientSession I don't see the new argument.

[ramant (doing other things), 2014/03/13 01:50:12]

Passed QuicSessionKey as argument to QuicClientSession and QuicClient.

[ramant (doing other things), 2014/03/13 01:50:12]

Added QuicSessionKey as argument to QuicClientSession and QuicClient (leaving it
to the callers to specify QuicSessionKey).

+                     crypto_config) {
 }
 
 QuicClientSession::~QuicClientSession() {