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

Unified Diff: net/quic/test_tools/quic_test_utils.cc

Issue 288313003: Land Recent QUIC Changes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src
Patch Set: implemented rch's comments Created 6 years, 7 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/test_tools/quic_test_utils.cc
diff --git a/net/quic/test_tools/quic_test_utils.cc b/net/quic/test_tools/quic_test_utils.cc
index 5e4d7f5b3d3ed17d67c6d0d255ab59b5a4815c48..f14dd7391c0b8ab4bb2503ea96e1b9c8b2b369e5 100644
--- a/net/quic/test_tools/quic_test_utils.cc
+++ b/net/quic/test_tools/quic_test_utils.cc
@@ -191,8 +191,7 @@ MockConnection::MockConnection(bool is_server)
IPEndPoint(TestPeerIPAddress(), kTestPort),
new testing::NiceMock<MockHelper>(),
new testing::NiceMock<MockPacketWriter>(),
- is_server, QuicSupportedVersions(),
- kInitialFlowControlWindowForTest),
+ is_server, QuicSupportedVersions()),
writer_(QuicConnectionPeer::GetWriter(this)),
helper_(helper()) {
}
@@ -202,8 +201,7 @@ MockConnection::MockConnection(IPEndPoint address,
: QuicConnection(kTestConnectionId, address,
new testing::NiceMock<MockHelper>(),
new testing::NiceMock<MockPacketWriter>(),
- is_server, QuicSupportedVersions(),
- kInitialFlowControlWindowForTest),
+ is_server, QuicSupportedVersions()),
writer_(QuicConnectionPeer::GetWriter(this)),
helper_(helper()) {
}
@@ -214,8 +212,7 @@ MockConnection::MockConnection(QuicConnectionId connection_id,
IPEndPoint(TestPeerIPAddress(), kTestPort),
new testing::NiceMock<MockHelper>(),
new testing::NiceMock<MockPacketWriter>(),
- is_server, QuicSupportedVersions(),
- kInitialFlowControlWindowForTest),
+ is_server, QuicSupportedVersions()),
writer_(QuicConnectionPeer::GetWriter(this)),
helper_(helper()) {
}
@@ -226,8 +223,7 @@ MockConnection::MockConnection(bool is_server,
IPEndPoint(TestPeerIPAddress(), kTestPort),
new testing::NiceMock<MockHelper>(),
new testing::NiceMock<MockPacketWriter>(),
- is_server, supported_versions,
- kInitialFlowControlWindowForTest),
+ is_server, supported_versions),
writer_(QuicConnectionPeer::GetWriter(this)),
helper_(helper()) {
}
@@ -266,7 +262,8 @@ bool PacketSavingConnection::SendOrQueuePacket(
}
MockSession::MockSession(QuicConnection* connection)
- : QuicSession(connection, DefaultQuicConfig()) {
+ : QuicSession(connection, kInitialFlowControlWindowForTest,
+ DefaultQuicConfig()) {
ON_CALL(*this, WritevData(_, _, _, _, _))
.WillByDefault(testing::Return(QuicConsumedData(0, false)));
}
@@ -274,11 +271,9 @@ MockSession::MockSession(QuicConnection* connection)
MockSession::~MockSession() {
}
-TestSession::TestSession(QuicConnection* connection,
- const QuicConfig& config)
- : QuicSession(connection, config),
- crypto_stream_(NULL) {
-}
+TestSession::TestSession(QuicConnection* connection, const QuicConfig& config)
+ : QuicSession(connection, kInitialFlowControlWindowForTest, config),
+ crypto_stream_(NULL) {}
TestSession::~TestSession() {}
@@ -292,7 +287,8 @@ QuicCryptoStream* TestSession::GetCryptoStream() {
TestClientSession::TestClientSession(QuicConnection* connection,
const QuicConfig& config)
- : QuicClientSessionBase(connection, config),
+ : QuicClientSessionBase(connection, kInitialFlowControlWindowForTest,
+ config),
crypto_stream_(NULL) {
EXPECT_CALL(*this, OnProofValid(_)).Times(AnyNumber());
}

Powered by Google App Engine
This is Rietveld 408576698