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

net/http/disk_cache_based_quic_server_info_unittest.cc

 // Copyright 2014 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/http/disk_cache_based_quic_server_info.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
 #include "base/message_loop/message_loop.h"
 #include "net/base/net_errors.h"
 #include "net/http/mock_http_cache.h"
 #include "net/quic/crypto/quic_server_info.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 // This is an empty transaction, needed to register the URL and the test mode.
 const MockTransaction kHostInfoTransaction = {
-  "quicserverinfo:https://www.google.com",
+  "quicserverinfo:https://www.google.com/443",
   "",
   base::Time(),
   "",
   net::LOAD_NORMAL,
   "",
   "",
   base::Time(),
   "",
   TEST_MODE_NORMAL,
   NULL,
   0
 };
 
 // Tests that we can delete a DiskCacheBasedQuicServerInfo object in a
 // completion callback for DiskCacheBasedQuicServerInfo::WaitForDataReady.
 TEST(DiskCacheBasedQuicServerInfo, DeleteInCallback) {
   // Use the blocking mock backend factory to force asynchronous completion
   // of quic_server_info->WaitForDataReady(), so that the callback will run.
   MockBlockingBackendFactory* factory = new MockBlockingBackendFactory();
   MockHttpCache cache(factory);
   scoped_ptr<net::QuicServerInfo> quic_server_info(
       new net::DiskCacheBasedQuicServerInfo("https://www.verisign.com",
+                                            443,

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

Should we have any test which demonstrates different info returned when the
ports differ?

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

Done.

                                             cache.http_cache()));
   quic_server_info->Start();
   net::TestCompletionCallback callback;
   int rv = quic_server_info->WaitForDataReady(callback.callback());
   EXPECT_EQ(net::ERR_IO_PENDING, rv);
   // Now complete the backend creation and let the callback run.
   factory->FinishCreation();
   EXPECT_EQ(net::OK, callback.GetResult(rv));
 }
 
 // Tests the basic logic of storing, retrieving and updating data.
 TEST(DiskCacheBasedQuicServerInfo, Update) {
   MockHttpCache cache;
   AddMockTransaction(&kHostInfoTransaction);
   net::TestCompletionCallback callback;
 
   scoped_ptr<net::QuicServerInfo> quic_server_info(
       new net::DiskCacheBasedQuicServerInfo("https://www.google.com",
+                                            443,
                                             cache.http_cache()));
   quic_server_info->Start();
   int rv = quic_server_info->WaitForDataReady(callback.callback());
   EXPECT_EQ(net::OK, callback.GetResult(rv));
 
   net::QuicServerInfo::State* state = quic_server_info->mutable_state();
   EXPECT_TRUE(state->certs.empty());
   const string server_config_a = "server_config_a";
   const string source_address_token_a = "source_address_token_a";
   const string server_config_sig_a = "server_config_sig_a";
   const string cert_a = "cert_a";
   const string cert_b = "cert_b";
 
   state->server_config = server_config_a;
   state->source_address_token = source_address_token_a;
   state->server_config_sig = server_config_sig_a;
   state->certs.push_back(cert_a);
   quic_server_info->Persist();
 
   // Wait until Persist() does the work.
   base::MessageLoop::current()->RunUntilIdle();
 
   // Open the stored QuicServerInfo.
   quic_server_info.reset(
       new net::DiskCacheBasedQuicServerInfo("https://www.google.com",
+                                            443,
                                             cache.http_cache()));
   quic_server_info->Start();
   rv = quic_server_info->WaitForDataReady(callback.callback());
   EXPECT_EQ(net::OK, callback.GetResult(rv));
 
   // And now update the data.
   state = quic_server_info->mutable_state();
   state->certs.push_back(cert_b);
 
   // Fail instead of DCHECKing double creates.
   cache.disk_cache()->set_double_create_check(false);
   quic_server_info->Persist();
   base::MessageLoop::current()->RunUntilIdle();
 
   // Verify that the state was updated.
   quic_server_info.reset(
       new net::DiskCacheBasedQuicServerInfo("https://www.google.com",
+                                            443,
                                             cache.http_cache()));
   quic_server_info->Start();
   rv = quic_server_info->WaitForDataReady(callback.callback());
   EXPECT_EQ(net::OK, callback.GetResult(rv));
 
   const net::QuicServerInfo::State& state1 = quic_server_info->state();
   EXPECT_TRUE(quic_server_info->IsDataReady());
   EXPECT_EQ(server_config_a, state1.server_config);
   EXPECT_EQ(source_address_token_a, state1.source_address_token);
   EXPECT_EQ(server_config_sig_a, state1.server_config_sig);
   EXPECT_EQ(2U, state1.certs.size());
   EXPECT_EQ(cert_a, state1.certs[0]);
   EXPECT_EQ(cert_b, state1.certs[1]);
 
   RemoveMockTransaction(&kHostInfoTransaction);
 }
 
 }  // namespace