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

net/http/http_cache.cc

Index: net/http/http_cache.cc
diff --git a/net/http/http_cache.cc b/net/http/http_cache.cc
index e4319e10a2d732005d6782c01429b92e033b6b54..c41bbe95d88a597186a7d8a09a537aa92425f288 100644
--- a/net/http/http_cache.cc
+++ b/net/http/http_cache.cc
@@ -275,8 +275,9 @@ class HttpCache::QuicServerInfoFactoryAdaptor : public QuicServerInfoFactory {
       : http_cache_(http_cache) {
   }
 
-  virtual QuicServerInfo* GetForHost(const std::string& hostname) OVERRIDE {
-    return new DiskCacheBasedQuicServerInfo(hostname, http_cache_);
+  virtual QuicServerInfo* GetForHostPortPair(

[wtc, 2014/03/11 01:41:39]

Nit: we can name this method "GetForServer". Then it's just a matter of how we
represent a server (by its hostname, by a host-port pair, or by
scheme://host:port).

[Ryan Hamilton, 2014/03/11 15:47:43]

Nice!  Or, if we're going to include scheme, we could also consider origin.

[ramant (doing other things), 2014/03/13 01:50:12]

Done.

[ramant (doing other things), 2014/03/13 01:50:12]

Acknowledged. Used GetServer per wtc's suggestion.

+      const HostPortPair& host_port_pair) OVERRIDE {
+    return new DiskCacheBasedQuicServerInfo(host_port_pair, http_cache_);
   }
 
  private: