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

Unified Diff: net/tools/quic/quic_spdy_client_stream_test.cc

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: Use QuicSessionKey as arg and delete server_hostname as arg 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_spdy_client_stream_test.cc
diff --git a/net/tools/quic/quic_spdy_client_stream_test.cc b/net/tools/quic/quic_spdy_client_stream_test.cc
index bad9080494defb2b81941775fa2dc1bd1da1185b..592037361806e80f05c5bb6e3329c26308eb17b7 100644
--- a/net/tools/quic/quic_spdy_client_stream_test.cc
+++ b/net/tools/quic/quic_spdy_client_stream_test.cc
@@ -30,8 +30,9 @@ class QuicSpdyClientStreamTest : public TestWithParam<QuicVersion> {
QuicSpdyClientStreamTest()
: connection_(new StrictMock<MockConnection>(
false, SupportedVersions(GetParam()))),
- session_("example.com", DefaultQuicConfig(), connection_,
- &crypto_config_),
+ session_key_("example.com", 80, false),
+ session_(session_key_, DefaultQuicConfig(), connection_,
+ &crypto_config_),
body_("hello world") {
crypto_config_.SetDefaults();
@@ -43,6 +44,7 @@ class QuicSpdyClientStreamTest : public TestWithParam<QuicVersion> {
}
StrictMock<MockConnection>* connection_;
+ QuicSessionKey session_key_;
QuicClientSession session_;
scoped_ptr<QuicSpdyClientStream> stream_;
BalsaHeaders headers_;

Powered by Google App Engine
This is Rietveld 408576698