| 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 8b340a86180159948d58be71cd0494a2253f4df5..f65db4ff5ceb189e06cbcd9abc8abf33029fa403 100644
|
| --- a/net/tools/flip_server/spdy_interface_test.cc
|
| +++ b/net/tools/flip_server/spdy_interface_test.cc
|
| @@ -54,10 +54,8 @@ class SpdyFramerVisitor : public BufferedSpdyFramerVisitorInterface {
|
| MOCK_METHOD3(OnSynReply, void(SpdyStreamId, bool, const SpdyHeaderBlock&));
|
| MOCK_METHOD3(OnHeaders, void(SpdyStreamId, bool, const SpdyHeaderBlock&));
|
| MOCK_METHOD3(OnDataFrameHeader, void(SpdyStreamId, size_t, bool));
|
| - MOCK_METHOD4(OnStreamFrameData, void(SpdyStreamId,
|
| - const char*,
|
| - size_t,
|
| - bool));
|
| + MOCK_METHOD4(OnStreamFrameData,
|
| + void(SpdyStreamId, const char*, size_t, bool));
|
| MOCK_METHOD1(OnSettings, void(bool clear_persisted));
|
| MOCK_METHOD3(OnSetting, void(SpdySettingsIds, uint8, uint32));
|
| MOCK_METHOD2(OnPing, void(SpdyPingId unique_id, bool is_ack));
|
| @@ -220,7 +218,8 @@ TEST_P(SpdySMProxyTest, OnSynStream_SPDY2) {
|
| scoped_ptr<MockSMInterface> mock_interface(new MockSMInterface);
|
| uint32 stream_id = 92;
|
| uint32 associated_id = 43;
|
| - std::string expected = "GET /path HTTP/1.0\r\n"
|
| + std::string expected =
|
| + "GET /path HTTP/1.0\r\n"
|
| "Host: 127.0.0.1\r\n"
|
| "hoge: fuga\r\n\r\n";
|
| SpdyHeaderBlock block;
|
| @@ -254,7 +253,8 @@ TEST_P(SpdySMProxyTest, OnSynStream) {
|
| scoped_ptr<MockSMInterface> mock_interface(new MockSMInterface);
|
| uint32 stream_id = 92;
|
| uint32 associated_id = 43;
|
| - std::string expected = "GET /path HTTP/1.1\r\n"
|
| + std::string expected =
|
| + "GET /path HTTP/1.1\r\n"
|
| "Host: 127.0.0.1\r\n"
|
| "foo: bar\r\n\r\n";
|
| SpdyHeaderBlock block;
|
| @@ -266,8 +266,7 @@ TEST_P(SpdySMProxyTest, OnSynStream) {
|
| StringSaver saver;
|
| {
|
| InSequence s;
|
| - EXPECT_CALL(*interface_,
|
| - FindOrMakeNewSMConnectionInterface(_, _))
|
| + EXPECT_CALL(*interface_, FindOrMakeNewSMConnectionInterface(_, _))
|
| .WillOnce(Return(mock_interface.get()));
|
| EXPECT_CALL(*mock_interface, SetStreamID(stream_id));
|
| EXPECT_CALL(*mock_interface, ProcessWriteInput(_, _))
|
| @@ -333,8 +332,7 @@ TEST_P(SpdySMProxyTest, OnStreamFrameData) {
|
| block["foo"] = "bar";
|
| {
|
| InSequence s;
|
| - EXPECT_CALL(*interface_,
|
| - FindOrMakeNewSMConnectionInterface(_, _))
|
| + EXPECT_CALL(*interface_, FindOrMakeNewSMConnectionInterface(_, _))
|
| .WillOnce(Return(mock_interface.get()));
|
| EXPECT_CALL(*mock_interface, SetStreamID(stream_id));
|
| EXPECT_CALL(*mock_interface, ProcessWriteInput(_, _)).Times(1);
|
| @@ -473,12 +471,11 @@ TEST_P(SpdySMProxyTest, SendErrorNotFound_SPDY2) {
|
| .WillOnce(SaveArg<2>(&actual_header_block));
|
| }
|
| EXPECT_CALL(checkpoint, Call(0));
|
| + EXPECT_CALL(*spdy_framer_visitor_, OnDataFrameHeader(stream_id, _, true));
|
| EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnDataFrameHeader(stream_id, _, true));
|
| - EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnStreamFrameData(stream_id, _, _, false)).Times(1)
|
| - .WillOnce(DoAll(SaveArg<1>(&actual_data),
|
| - SaveArg<2>(&actual_size)));
|
| + OnStreamFrameData(stream_id, _, _, false))
|
| + .Times(1)
|
| + .WillOnce(DoAll(SaveArg<1>(&actual_data), SaveArg<2>(&actual_size)));
|
| EXPECT_CALL(*spdy_framer_visitor_,
|
| OnStreamFrameData(stream_id, NULL, 0, true)).Times(1);
|
| }
|
| @@ -515,21 +512,18 @@ TEST_P(SpdySMProxyTest, SendErrorNotFound) {
|
| {
|
| InSequence s;
|
| if (GetParam() < SPDY4) {
|
| - EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnSynReply(stream_id, false, _))
|
| + 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, _))
|
| + EXPECT_CALL(*spdy_framer_visitor_, OnHeaders(stream_id, false, _))
|
| .WillOnce(SaveArg<2>(&actual_header_block));
|
| }
|
| EXPECT_CALL(checkpoint, Call(0));
|
| + EXPECT_CALL(*spdy_framer_visitor_, OnDataFrameHeader(stream_id, _, true));
|
| EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnDataFrameHeader(stream_id, _, true));
|
| - EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnStreamFrameData(stream_id, _, _, false)).Times(1)
|
| - .WillOnce(DoAll(SaveArg<1>(&actual_data),
|
| - SaveArg<2>(&actual_size)));
|
| + OnStreamFrameData(stream_id, _, _, false))
|
| + .Times(1)
|
| + .WillOnce(DoAll(SaveArg<1>(&actual_data), SaveArg<2>(&actual_size)));
|
| EXPECT_CALL(*spdy_framer_visitor_,
|
| OnStreamFrameData(stream_id, NULL, 0, true)).Times(1);
|
| }
|
| @@ -702,8 +696,7 @@ TEST_P(SpdySMProxyTest, SendDataFrame) {
|
|
|
| {
|
| InSequence s;
|
| - EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnDataFrameHeader(stream_id, _, false));
|
| + EXPECT_CALL(*spdy_framer_visitor_, OnDataFrameHeader(stream_id, _, false));
|
| EXPECT_CALL(*spdy_framer_visitor_,
|
| OnStreamFrameData(stream_id, _, _, false))
|
| .WillOnce(DoAll(SaveArg<1>(&actual_data), SaveArg<2>(&actual_size)));
|
| @@ -727,12 +720,11 @@ TEST_P(SpdySMProxyTest, SendLongDataFrame) {
|
| {
|
| InSequence s;
|
| for (int i = 0; i < 3; ++i) {
|
| - EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnDataFrameHeader(stream_id, _, false));
|
| - EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnStreamFrameData(stream_id, _, _, false))
|
| - .WillOnce(DoAll(SaveArg<1>(&actual_data),
|
| - SaveArg<2>(&actual_size)));
|
| + EXPECT_CALL(*spdy_framer_visitor_,
|
| + OnDataFrameHeader(stream_id, _, false));
|
| + EXPECT_CALL(*spdy_framer_visitor_,
|
| + OnStreamFrameData(stream_id, _, _, false))
|
| + .WillOnce(DoAll(SaveArg<1>(&actual_data), SaveArg<2>(&actual_size)));
|
| }
|
| }
|
|
|
| @@ -856,12 +848,11 @@ TEST_P(SpdySMServerTest, NewStreamError) {
|
| .WillOnce(SaveArg<2>(&actual_header_block));
|
| }
|
| EXPECT_CALL(checkpoint, Call(0));
|
| + EXPECT_CALL(*spdy_framer_visitor_, OnDataFrameHeader(stream_id, _, true));
|
| EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnDataFrameHeader(stream_id, _, true));
|
| - EXPECT_CALL(*spdy_framer_visitor_,
|
| - OnStreamFrameData(stream_id, _, _, false)).Times(1)
|
| - .WillOnce(DoAll(SaveArg<1>(&actual_data),
|
| - SaveArg<2>(&actual_size)));
|
| + OnStreamFrameData(stream_id, _, _, false))
|
| + .Times(1)
|
| + .WillOnce(DoAll(SaveArg<1>(&actual_data), SaveArg<2>(&actual_size)));
|
| EXPECT_CALL(*spdy_framer_visitor_,
|
| OnStreamFrameData(stream_id, NULL, 0, true)).Times(1);
|
| }
|
|
|