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

net/quic/test_tools/mock_crypto_client_stream_factory.cc

Index: net/quic/test_tools/mock_crypto_client_stream_factory.cc
diff --git a/net/quic/test_tools/mock_crypto_client_stream_factory.cc b/net/quic/test_tools/mock_crypto_client_stream_factory.cc
index 671a87210e58da38bb9f88e79b42940825119842..18923a61bb09e8991f57304b18d00579b699c68e 100644
--- a/net/quic/test_tools/mock_crypto_client_stream_factory.cc
+++ b/net/quic/test_tools/mock_crypto_client_stream_factory.cc
@@ -20,10 +20,11 @@ MockCryptoClientStreamFactory::MockCryptoClientStreamFactory()
 QuicCryptoClientStream*
 MockCryptoClientStreamFactory::CreateQuicCryptoClientStream(
     const string& server_hostname,
+    uint16 server_port,
     QuicSession* session,
     QuicCryptoClientConfig* crypto_config) {
-  last_stream_ = new MockCryptoClientStream(
-      server_hostname, session, crypto_config, handshake_mode_, ssl_info_);
+  last_stream_ = new MockCryptoClientStream(server_hostname, server_port,

[Ryan Hamilton, 2014/03/10 20:57:33]

nit: I think this wrapping is unusual.  I think that either server_hostname
should be on the next line, or else session should line up with server_hostname?

[ramant (doing other things), 2014/03/11 00:48:50]

Done.

+      session, crypto_config, handshake_mode_, ssl_info_);
   return last_stream_;
 }