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

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

Issue 2820263005: In QUIC version >= 38, enables random padding of size [1, 256] (Closed)
Patch Set: Created 3 years, 8 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_simple_server_session_test.cc
diff --git a/net/tools/quic/quic_simple_server_session_test.cc b/net/tools/quic/quic_simple_server_session_test.cc
index 4cc03290a4f10a440e0ba43027ac1868ac5622aa..5f72d05a57080ff27b8a0be61ead6db4de0b5872 100644
--- a/net/tools/quic/quic_simple_server_session_test.cc
+++ b/net/tools/quic/quic_simple_server_session_test.cc
@@ -123,7 +123,7 @@ class MockQuicConnectionWithSendStreamData : public MockQuicConnection {
QuicStreamId id,
QuicIOVector iov,
QuicStreamOffset offset,
- bool fin,
+ StreamSendingState state,
QuicReferenceCountedPointer<QuicAckListenerInterface> ack_listener));
};
@@ -490,7 +490,7 @@ class QuicSimpleServerSessionServerPushTest
// Since flow control window is smaller than response body, not the
// whole body will be sent.
if (!session_->force_hol_blocking()) {
- EXPECT_CALL(*connection_, SendStreamData(stream_id, _, 0, false, _))
+ EXPECT_CALL(*connection_, SendStreamData(stream_id, _, 0, NO_FIN, _))
.WillOnce(Return(
QuicConsumedData(kStreamFlowControlWindowSize, false)));
EXPECT_CALL(*connection_, SendBlocked(stream_id));
@@ -499,7 +499,7 @@ class QuicSimpleServerSessionServerPushTest
// HTTP/2 DATA frames within the headers stream. HTTP/2
// DATA frames are limited to a max size of 16KB, so the
// 64KB body will be fragemented into four DATA frames.
- EXPECT_CALL(*connection_, SendStreamData(_, _, _, false, _))
+ EXPECT_CALL(*connection_, SendStreamData(_, _, _, NO_FIN, _))
.Times(body_size / 16384)
.WillOnce(Return(QuicConsumedData(9 + 16394, false)))
.WillOnce(Return(QuicConsumedData(9 + 16394, false)))
@@ -548,7 +548,7 @@ TEST_P(QuicSimpleServerSessionServerPushTest,
EXPECT_CALL(*session_, WriteHeadersMock(next_out_going_stream_id, _, false,
kDefaultPriority, _));
EXPECT_CALL(*connection_,
- SendStreamData(next_out_going_stream_id, _, 0, false, _))
+ SendStreamData(next_out_going_stream_id, _, 0, NO_FIN, _))
.WillOnce(Return(QuicConsumedData(kStreamFlowControlWindowSize, false)));
EXPECT_CALL(*connection_, SendBlocked(next_out_going_stream_id));
session_->StreamDraining(2);
@@ -585,7 +585,7 @@ TEST_P(QuicSimpleServerSessionServerPushTest,
InSequence s;
EXPECT_CALL(*session_, WriteHeadersMock(stream_not_reset, _, false,
kDefaultPriority, _));
- EXPECT_CALL(*connection_, SendStreamData(stream_not_reset, _, 0, false, _))
+ EXPECT_CALL(*connection_, SendStreamData(stream_not_reset, _, 0, NO_FIN, _))
.WillOnce(Return(QuicConsumedData(kStreamFlowControlWindowSize, false)));
EXPECT_CALL(*connection_, SendBlocked(stream_not_reset));
EXPECT_CALL(*session_,
@@ -614,7 +614,7 @@ TEST_P(QuicSimpleServerSessionServerPushTest,
SendRstStream(stream_got_reset, QUIC_RST_ACKNOWLEDGEMENT, _));
EXPECT_CALL(*session_,
WriteHeadersMock(stream_to_open, _, false, kDefaultPriority, _));
- EXPECT_CALL(*connection_, SendStreamData(stream_to_open, _, 0, false, _))
+ EXPECT_CALL(*connection_, SendStreamData(stream_to_open, _, 0, NO_FIN, _))
.WillOnce(Return(QuicConsumedData(kStreamFlowControlWindowSize, false)));
EXPECT_CALL(*connection_, SendBlocked(stream_to_open));

Powered by Google App Engine
This is Rietveld 408576698