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

Unified Diff: net/tools/flip_server/spdy_interface_test.cc

Issue 723343002: Update from https://crrev.com/304121 (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years, 1 month 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
« no previous file with comments | « net/tools/flip_server/spdy_interface.cc ('k') | net/tools/quic/end_to_end_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/flip_server/spdy_interface_test.cc
diff --git a/net/tools/flip_server/spdy_interface_test.cc b/net/tools/flip_server/spdy_interface_test.cc
index 9e66d0c37cc4b0b5bcfc1069a901cae40da8ddaa..09909e2953e43f386b18cf5a138849d055080d37 100644
--- a/net/tools/flip_server/spdy_interface_test.cc
+++ b/net/tools/flip_server/spdy_interface_test.cc
@@ -52,7 +52,12 @@ class SpdyFramerVisitor : public BufferedSpdyFramerVisitorInterface {
bool,
const SpdyHeaderBlock&));
MOCK_METHOD3(OnSynReply, void(SpdyStreamId, bool, const SpdyHeaderBlock&));
- MOCK_METHOD3(OnHeaders, void(SpdyStreamId, bool, const SpdyHeaderBlock&));
+ MOCK_METHOD5(OnHeaders,
+ void(SpdyStreamId,
+ bool,
+ SpdyPriority,
+ bool,
+ const SpdyHeaderBlock&));
MOCK_METHOD3(OnDataFrameHeader, void(SpdyStreamId, size_t, bool));
MOCK_METHOD4(OnStreamFrameData, void(SpdyStreamId,
const char*,
@@ -471,8 +476,9 @@ TEST_P(SpdySMProxyTest, SendErrorNotFound_SPDY2) {
EXPECT_CALL(*spdy_framer_visitor_, OnSynReply(stream_id, false, _))
.WillOnce(SaveArg<2>(&actual_header_block));
} else {
- EXPECT_CALL(*spdy_framer_visitor_, OnHeaders(stream_id, false, _))
- .WillOnce(SaveArg<2>(&actual_header_block));
+ EXPECT_CALL(*spdy_framer_visitor_,
+ OnHeaders(stream_id, false, 0, false, _))
+ .WillOnce(SaveArg<4>(&actual_header_block));
}
EXPECT_CALL(checkpoint, Call(0));
EXPECT_CALL(*spdy_framer_visitor_,
@@ -522,8 +528,8 @@ TEST_P(SpdySMProxyTest, SendErrorNotFound) {
.WillOnce(SaveArg<2>(&actual_header_block));
} else {
EXPECT_CALL(*spdy_framer_visitor_,
- OnHeaders(stream_id, false, _))
- .WillOnce(SaveArg<2>(&actual_header_block));
+ OnHeaders(stream_id, false, 0, false, _))
+ .WillOnce(SaveArg<4>(&actual_header_block));
}
EXPECT_CALL(checkpoint, Call(0));
EXPECT_CALL(*spdy_framer_visitor_,
@@ -641,8 +647,9 @@ TEST_P(SpdySMProxyTest, SendSynReply_SPDY2) {
EXPECT_CALL(*spdy_framer_visitor_, OnSynReply(stream_id, false, _))
.WillOnce(SaveArg<2>(&actual_header_block));
} else {
- EXPECT_CALL(*spdy_framer_visitor_, OnHeaders(stream_id, false, _))
- .WillOnce(SaveArg<2>(&actual_header_block));
+ EXPECT_CALL(*spdy_framer_visitor_,
+ OnHeaders(stream_id, false, 0, false, _))
+ .WillOnce(SaveArg<4>(&actual_header_block));
}
}
@@ -677,8 +684,9 @@ TEST_P(SpdySMProxyTest, SendSynReply) {
EXPECT_CALL(*spdy_framer_visitor_, OnSynReply(stream_id, false, _))
.WillOnce(SaveArg<2>(&actual_header_block));
} else {
- EXPECT_CALL(*spdy_framer_visitor_, OnHeaders(stream_id, false, _))
- .WillOnce(SaveArg<2>(&actual_header_block));
+ EXPECT_CALL(*spdy_framer_visitor_,
+ OnHeaders(stream_id, false, 0, false, _))
+ .WillOnce(SaveArg<4>(&actual_header_block));
}
}
@@ -854,8 +862,9 @@ TEST_P(SpdySMServerTest, NewStreamError) {
EXPECT_CALL(*spdy_framer_visitor_, OnSynReply(stream_id, false, _))
.WillOnce(SaveArg<2>(&actual_header_block));
} else {
- EXPECT_CALL(*spdy_framer_visitor_, OnHeaders(stream_id, false, _))
- .WillOnce(SaveArg<2>(&actual_header_block));
+ EXPECT_CALL(*spdy_framer_visitor_,
+ OnHeaders(stream_id, false, 0, false, _))
+ .WillOnce(SaveArg<4>(&actual_header_block));
}
EXPECT_CALL(checkpoint, Call(0));
EXPECT_CALL(*spdy_framer_visitor_,
« no previous file with comments | « net/tools/flip_server/spdy_interface.cc ('k') | net/tools/quic/end_to_end_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698