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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/tools/quic/quic_spdy_client_stream.h" 5 #include "net/tools/quic/quic_spdy_client_stream.h"
6 6
7 #include "base/strings/string_number_conversions.h" 7 #include "base/strings/string_number_conversions.h"
8 #include "net/quic/quic_utils.h" 8 #include "net/quic/quic_utils.h"
9 #include "net/quic/test_tools/quic_test_utils.h" 9 #include "net/quic/test_tools/quic_test_utils.h"
10 #include "net/tools/epoll_server/epoll_server.h" 10 #include "net/tools/epoll_server/epoll_server.h"
(...skipping 12 matching lines...) Expand all
23 namespace net { 23 namespace net {
24 namespace tools { 24 namespace tools {
25 namespace test { 25 namespace test {
26 namespace { 26 namespace {
27 27
28 class QuicSpdyClientStreamTest : public TestWithParam<QuicVersion> { 28 class QuicSpdyClientStreamTest : public TestWithParam<QuicVersion> {
29 public: 29 public:
30 QuicSpdyClientStreamTest() 30 QuicSpdyClientStreamTest()
31 : connection_(new StrictMock<MockConnection>( 31 : connection_(new StrictMock<MockConnection>(
32 false, SupportedVersions(GetParam()))), 32 false, SupportedVersions(GetParam()))),
33 session_("example.com", DefaultQuicConfig(), connection_, 33 session_key_("example.com", 80, false),
34 &crypto_config_), 34 session_(session_key_, DefaultQuicConfig(), connection_,
35 &crypto_config_),
35 body_("hello world") { 36 body_("hello world") {
36 crypto_config_.SetDefaults(); 37 crypto_config_.SetDefaults();
37 38
38 headers_.SetResponseFirstlineFromStringPieces("HTTP/1.1", "200", "Ok"); 39 headers_.SetResponseFirstlineFromStringPieces("HTTP/1.1", "200", "Ok");
39 headers_.ReplaceOrAppendHeader("content-length", "11"); 40 headers_.ReplaceOrAppendHeader("content-length", "11");
40 41
41 headers_string_ = SpdyUtils::SerializeResponseHeaders(headers_); 42 headers_string_ = SpdyUtils::SerializeResponseHeaders(headers_);
42 stream_.reset(new QuicSpdyClientStream(3, &session_)); 43 stream_.reset(new QuicSpdyClientStream(3, &session_));
43 } 44 }
44 45
45 StrictMock<MockConnection>* connection_; 46 StrictMock<MockConnection>* connection_;
47 QuicSessionKey session_key_;
46 QuicClientSession session_; 48 QuicClientSession session_;
47 scoped_ptr<QuicSpdyClientStream> stream_; 49 scoped_ptr<QuicSpdyClientStream> stream_;
48 BalsaHeaders headers_; 50 BalsaHeaders headers_;
49 string headers_string_; 51 string headers_string_;
50 string body_; 52 string body_;
51 QuicCryptoClientConfig crypto_config_; 53 QuicCryptoClientConfig crypto_config_;
52 }; 54 };
53 55
54 INSTANTIATE_TEST_CASE_P(Tests, QuicSpdyClientStreamTest, 56 INSTANTIATE_TEST_CASE_P(Tests, QuicSpdyClientStreamTest,
55 ::testing::ValuesIn(QuicSupportedVersions())); 57 ::testing::ValuesIn(QuicSupportedVersions()));
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 95
94 EXPECT_FALSE(stream_->write_side_closed()); 96 EXPECT_FALSE(stream_->write_side_closed());
95 EXPECT_TRUE(stream_->OnStreamFrame(frame)); 97 EXPECT_TRUE(stream_->OnStreamFrame(frame));
96 EXPECT_TRUE(stream_->write_side_closed()); 98 EXPECT_TRUE(stream_->write_side_closed());
97 } 99 }
98 100
99 } // namespace 101 } // namespace
100 } // namespace test 102 } // namespace test
101 } // namespace tools 103 } // namespace tools
102 } // namespace net 104 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698