Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(340)

Unified Diff: net/quic/crypto/quic_server_info.h

Issue 192583004: QUIC - use QuicSessionKey tuple (host, port, is_https) instead of server_hostname (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge with TOT Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/crypto/quic_server_info.h
diff --git a/net/quic/crypto/quic_server_info.h b/net/quic/crypto/quic_server_info.h
index 853504ae70f96882c3d1d5e66165097892554dc3..51533ec0c3b87da20ff781ba1610eeafc57e5a73 100644
--- a/net/quic/crypto/quic_server_info.h
+++ b/net/quic/crypto/quic_server_info.h
@@ -13,6 +13,7 @@
#include "base/time/time.h"
#include "net/base/completion_callback.h"
#include "net/base/net_export.h"
+#include "net/quic/quic_session_key.h"
namespace net {
@@ -24,7 +25,7 @@ class X509Certificate;
// crypto config.
class NET_EXPORT_PRIVATE QuicServerInfo {
public:
- QuicServerInfo(const std::string& hostname);
+ QuicServerInfo(const QuicSessionKey& server_key);
virtual ~QuicServerInfo();
// Start will commence the lookup. This must be called before any other
@@ -92,17 +93,18 @@ class NET_EXPORT_PRIVATE QuicServerInfo {
// SerializeInner is a helper function for Serialize.
std::string SerializeInner() const;
- // This is the QUIC server hostname for which we restore the crypto_config.
- const std::string hostname_;
+ // This is the QUIC server (hostname, port, is_https) tuple for which we
+ // restore the crypto_config.
+ const QuicSessionKey server_key_;
};
class QuicServerInfoFactory {
public:
virtual ~QuicServerInfoFactory();
- // GetForHost returns a fresh, allocated QuicServerInfo for the given
- // hostname or NULL on failure.
- virtual QuicServerInfo* GetForHost(const std::string& hostname) = 0;
+ // GetForServer returns a fresh, allocated QuicServerInfo for the given
+ // |server_key| or NULL on failure.
+ virtual QuicServerInfo* GetForServer(const QuicSessionKey& server_key) = 0;
};
} // namespace net

Powered by Google App Engine
This is Rietveld 408576698