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

net/quic/test_tools/mock_crypto_client_stream_factory.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/test_tools/mock_crypto_client_stream_factory.h"
 
 #include "base/lazy_instance.h"
 #include "net/quic/quic_crypto_client_stream.h"
 
 using std::string;
 
 namespace net {
 
 MockCryptoClientStreamFactory::MockCryptoClientStreamFactory()
     : handshake_mode_(MockCryptoClientStream::CONFIRM_HANDSHAKE),
       last_stream_(NULL),
       ssl_info_(NULL) {
 }
 
 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_;
 }
 
 }  // namespace net