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

net/tools/quic/test_tools/quic_test_client.cc

Index: net/tools/quic/test_tools/quic_test_client.cc
diff --git a/net/tools/quic/test_tools/quic_test_client.cc b/net/tools/quic/test_tools/quic_test_client.cc
index 8cfeefb5f5b2028c983396c857f6e1aa3517d6af..cea717b37842f0896f5c143f005f82300b6cf00b 100644
--- a/net/tools/quic/test_tools/quic_test_client.cc
+++ b/net/tools/quic/test_tools/quic_test_client.cc
@@ -291,8 +291,9 @@ const string& QuicTestClient::cert_common_name() const {
 QuicTagValueMap QuicTestClient::GetServerConfig() const {
   net::QuicCryptoClientConfig* config =
       QuicClientPeer::GetCryptoConfig(client_.get());
+  // Defaulting server port to 9999.
   net::QuicCryptoClientConfig::CachedState* state =
-      config->LookupOrCreate(client_->server_hostname());
+      config->LookupOrCreate(client_->server_hostname(), 9999);

[ramant (doing other things), 2014/03/10 20:31:46]

We are not passing port for QuiTestClient and this code doesn't use chrome's
disk_cache (quic_server_info_factory is not initialized). Should we default to
port 0??

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

Using the port from client's server_address.

   const net::CryptoHandshakeMessage* handshake_msg = state->GetServerConfig();
   if (handshake_msg != NULL) {
     return handshake_msg->tag_value_map();