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

Unified Diff: net/tools/quic/quic_client.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/tools/quic/quic_client.h
diff --git a/net/tools/quic/quic_client.h b/net/tools/quic/quic_client.h
index aec66798ab70c7da6693e0206d60bf7ea27dc106..8ea37ea67d05484c50047a3e726e62af02ca0c6a 100644
--- a/net/tools/quic/quic_client.h
+++ b/net/tools/quic/quic_client.h
@@ -25,6 +25,7 @@
namespace net {
class ProofVerifier;
+class QuicSessionKey;
namespace tools {
@@ -47,11 +48,11 @@ class QuicClient : public EpollCallbackInterface,
};
QuicClient(IPEndPoint server_address,
- const string& server_hostname,
+ const QuicSessionKey& server_key,
const QuicVersionVector& supported_versions,
bool print_response);
QuicClient(IPEndPoint server_address,
- const std::string& server_hostname,
+ const QuicSessionKey& server_key,
const QuicConfig& config,
const QuicVersionVector& supported_versions);
@@ -134,13 +135,11 @@ class QuicClient : public EpollCallbackInterface,
int fd() { return fd_; }
- string server_hostname() {
- return server_hostname_;
- }
+ const QuicSessionKey& server_key() const { return server_key_; }
// This should only be set before the initial Connect()
- void set_server_hostname(const string& hostname) {
- server_hostname_ = hostname;
+ void set_server_key(const QuicSessionKey& server_key) {
+ server_key_ = server_key;
}
// SetProofVerifier sets the ProofVerifier that will be used to verify the
@@ -181,8 +180,8 @@ class QuicClient : public EpollCallbackInterface,
// Address of the server.
const IPEndPoint server_address_;
- // Hostname of the server. This may be a DNS name or an IP address literal.
- std::string server_hostname_;
+ // |server_key_| is a tuple (hostname, port, is_https) of the server.
+ QuicSessionKey server_key_;
// config_ and crypto_config_ contain configuration and cached state about
// servers.

Powered by Google App Engine
This is Rietveld 408576698