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

net/quic/quic_client_session.h

Index: net/quic/quic_client_session.h
diff --git a/net/quic/quic_client_session.h b/net/quic/quic_client_session.h
index f56a9b652ae5f3cd07f0b5804f49dc78b5441d76..2b2ffd2fe64675d853567dfa2fef65a7b47da2af 100644
--- a/net/quic/quic_client_session.h
+++ b/net/quic/quic_client_session.h
@@ -29,6 +29,7 @@ class DatagramClientSocket;
 class QuicConnectionHelper;
 class QuicCryptoClientStreamFactory;
 class QuicDefaultPacketWriter;
+class QuicSessionKey;
 class QuicStreamFactory;
 class SSLInfo;
 
@@ -92,7 +93,7 @@ class NET_EXPORT_PRIVATE QuicClientSession : public QuicSession {
                     QuicStreamFactory* stream_factory,
                     scoped_ptr<QuicServerInfo> server_info,
                     QuicCryptoClientStreamFactory* crypto_client_stream_factory,
-                    const std::string& server_hostname,
+                    const QuicSessionKey& server_key,

[wtc, 2014/03/17 17:13:21]

Nit: server_info and server_key probably should be listed next to each other.

[ramant (doing other things), 2014/03/17 18:27:36]

Fixed it in https://codereview.chromium.org/202143004

Done.

                     const QuicConfig& config,
                     QuicCryptoClientConfig* crypto_config,
                     NetLog* net_log);