OLD | NEW |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 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/spdy/spdy_headers_block_parser.h" | 5 #include "net/spdy/spdy_headers_block_parser.h" |
6 | 6 |
7 #include <memory> | 7 #include <memory> |
8 #include <string> | 8 #include <string> |
9 | 9 |
10 #include "base/strings/string_number_conversions.h" | 10 #include "base/strings/string_number_conversions.h" |
11 #include "base/sys_byteorder.h" | 11 #include "base/sys_byteorder.h" |
| 12 #include "net/spdy/spdy_flags.h" |
12 #include "net/spdy/spdy_test_utils.h" | 13 #include "net/spdy/spdy_test_utils.h" |
13 #include "net/test/gtest_util.h" | 14 #include "net/test/gtest_util.h" |
14 #include "testing/gmock/include/gmock/gmock.h" | 15 #include "testing/gmock/include/gmock/gmock.h" |
15 #include "testing/gtest/include/gtest/gtest.h" | 16 #include "testing/gtest/include/gtest/gtest.h" |
16 | 17 |
| 18 using testing::_; |
| 19 |
17 namespace net { | 20 namespace net { |
18 | 21 |
19 using base::IntToString; | 22 using base::IntToString; |
20 using base::StringPiece; | 23 using base::StringPiece; |
21 using base::UintToString; | 24 using base::UintToString; |
22 using std::string; | 25 using std::string; |
23 | 26 |
24 // A mock the handler class to check that we parse out the correct headers | 27 // A mock the handler class to check that we parse out the correct headers |
25 // and call the callback methods when we should. | 28 // and call the callback methods when we should. |
26 class MockSpdyHeadersHandler : public SpdyHeadersHandlerInterface { | 29 class MockSpdyHeadersHandler : public SpdyHeadersHandlerInterface { |
27 public: | 30 public: |
28 MOCK_METHOD0(OnHeaderBlockStart, void()); | 31 MOCK_METHOD0(OnHeaderBlockStart, void()); |
29 MOCK_METHOD1(OnHeaderBlockEnd, void(size_t bytes)); | 32 MOCK_METHOD1(OnHeaderBlockEnd, void(size_t bytes)); |
| 33 MOCK_METHOD2(OnHeaderBlockEnd, void(size_t bytes, size_t compressed_bytes)); |
30 MOCK_METHOD2(OnHeader, void(StringPiece key, StringPiece value)); | 34 MOCK_METHOD2(OnHeader, void(StringPiece key, StringPiece value)); |
31 }; | 35 }; |
32 | 36 |
33 class SpdyHeadersBlockParserTest : | 37 class SpdyHeadersBlockParserTest : |
34 public ::testing::TestWithParam<SpdyMajorVersion> { | 38 public ::testing::TestWithParam<SpdyMajorVersion> { |
35 public: | 39 public: |
36 ~SpdyHeadersBlockParserTest() override {} | 40 ~SpdyHeadersBlockParserTest() override {} |
37 | 41 |
38 protected: | 42 protected: |
39 void SetUp() override { | 43 void SetUp() override { |
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
111 // handling a headers block. | 115 // handling a headers block. |
112 EXPECT_EQ(spdy_version_, parser_->spdy_version()); | 116 EXPECT_EQ(spdy_version_, parser_->spdy_version()); |
113 | 117 |
114 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); | 118 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); |
115 string expect_key = kBaseKey + IntToString(0); | 119 string expect_key = kBaseKey + IntToString(0); |
116 string expect_value = kBaseValue + IntToString(0); | 120 string expect_value = kBaseValue + IntToString(0); |
117 EXPECT_CALL(handler_, OnHeader(StringPiece(expect_key), | 121 EXPECT_CALL(handler_, OnHeader(StringPiece(expect_key), |
118 StringPiece(expect_value))).Times(1); | 122 StringPiece(expect_value))).Times(1); |
119 | 123 |
120 string headers(CreateHeaders(1, false)); | 124 string headers(CreateHeaders(1, false)); |
121 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); | 125 if (FLAGS_chromium_http2_flag_log_compressed_size) { |
122 | 126 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length(), _)).Times(1); |
| 127 } else { |
| 128 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); |
| 129 } |
123 EXPECT_TRUE(parser_-> | 130 EXPECT_TRUE(parser_-> |
124 HandleControlFrameHeadersData(1, headers.c_str(), headers.length())); | 131 HandleControlFrameHeadersData(1, headers.c_str(), headers.length())); |
125 EXPECT_EQ(SpdyHeadersBlockParser::NO_PARSER_ERROR, parser_->get_error()); | 132 EXPECT_EQ(SpdyHeadersBlockParser::NO_PARSER_ERROR, parser_->get_error()); |
126 } | 133 } |
127 | 134 |
128 TEST_P(SpdyHeadersBlockParserTest, NullsSupportedTest) { | 135 TEST_P(SpdyHeadersBlockParserTest, NullsSupportedTest) { |
129 // Sanity test, verify that we parse out correctly a block with | 136 // Sanity test, verify that we parse out correctly a block with |
130 // a single key-value pair when the key and value contain null charecters. | 137 // a single key-value pair when the key and value contain null charecters. |
131 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); | 138 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); |
132 | 139 |
133 string expect_key = kBaseKey + string("\0", 1) + IntToString(0); | 140 string expect_key = kBaseKey + string("\0", 1) + IntToString(0); |
134 string expect_value = kBaseValue + string("\0", 1) + IntToString(0); | 141 string expect_value = kBaseValue + string("\0", 1) + IntToString(0); |
135 EXPECT_CALL(handler_, OnHeader(StringPiece(expect_key), | 142 EXPECT_CALL(handler_, OnHeader(StringPiece(expect_key), |
136 StringPiece(expect_value))).Times(1); | 143 StringPiece(expect_value))).Times(1); |
137 | 144 |
138 string headers(CreateHeaders(1, true)); | 145 string headers(CreateHeaders(1, true)); |
139 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); | 146 if (FLAGS_chromium_http2_flag_log_compressed_size) { |
| 147 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length(), _)).Times(1); |
| 148 } else { |
| 149 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); |
| 150 } |
140 | 151 |
141 EXPECT_TRUE(parser_-> | 152 EXPECT_TRUE(parser_-> |
142 HandleControlFrameHeadersData(1, headers.c_str(), headers.length())); | 153 HandleControlFrameHeadersData(1, headers.c_str(), headers.length())); |
143 EXPECT_EQ(SpdyHeadersBlockParser::NO_PARSER_ERROR, parser_->get_error()); | 154 EXPECT_EQ(SpdyHeadersBlockParser::NO_PARSER_ERROR, parser_->get_error()); |
144 } | 155 } |
145 | 156 |
146 TEST_P(SpdyHeadersBlockParserTest, MultipleBlocksAndHeadersWithPartialData) { | 157 TEST_P(SpdyHeadersBlockParserTest, MultipleBlocksAndHeadersWithPartialData) { |
147 testing::InSequence s; | 158 testing::InSequence s; |
148 | 159 |
149 // CreateHeaders is deterministic; we can call it once for the whole test. | 160 // CreateHeaders is deterministic; we can call it once for the whole test. |
150 string headers(CreateHeaders(kNumHeadersInBlock, false)); | 161 string headers(CreateHeaders(kNumHeadersInBlock, false)); |
151 | 162 |
152 // The mock doesn't retain storage of arguments, so keep them in scope. | 163 // The mock doesn't retain storage of arguments, so keep them in scope. |
153 std::vector<string> retained_arguments; | 164 std::vector<string> retained_arguments; |
154 for (int i = 0; i < kNumHeadersInBlock; i++) { | 165 for (int i = 0; i < kNumHeadersInBlock; i++) { |
155 retained_arguments.push_back(kBaseKey + IntToString(i)); | 166 retained_arguments.push_back(kBaseKey + IntToString(i)); |
156 retained_arguments.push_back(kBaseValue + IntToString(i)); | 167 retained_arguments.push_back(kBaseValue + IntToString(i)); |
157 } | 168 } |
158 // For each block we expect to parse out the headers in order. | 169 // For each block we expect to parse out the headers in order. |
159 for (int i = 0; i < kNumHeaderBlocks; i++) { | 170 for (int i = 0; i < kNumHeaderBlocks; i++) { |
160 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); | 171 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); |
161 for (int j = 0; j < kNumHeadersInBlock; j++) { | 172 for (int j = 0; j < kNumHeadersInBlock; j++) { |
162 EXPECT_CALL(handler_, OnHeader( | 173 EXPECT_CALL(handler_, OnHeader( |
163 StringPiece(retained_arguments[2 * j]), | 174 StringPiece(retained_arguments[2 * j]), |
164 StringPiece(retained_arguments[2 * j + 1]))).Times(1); | 175 StringPiece(retained_arguments[2 * j + 1]))).Times(1); |
165 } | 176 } |
166 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); | 177 if (FLAGS_chromium_http2_flag_log_compressed_size) { |
| 178 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length(), _)).Times(1); |
| 179 } else { |
| 180 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); |
| 181 } |
167 } | 182 } |
168 // Parse the header blocks, feeding the parser one byte at a time. | 183 // Parse the header blocks, feeding the parser one byte at a time. |
169 for (int i = 1; i <= kNumHeaderBlocks; i++) { | 184 for (int i = 1; i <= kNumHeaderBlocks; i++) { |
170 for (string::iterator it = headers.begin(); it != headers.end(); ++it) { | 185 for (string::iterator it = headers.begin(); it != headers.end(); ++it) { |
171 if ((it + 1) == headers.end()) { | 186 if ((it + 1) == headers.end()) { |
172 // Last byte completes the block. | 187 // Last byte completes the block. |
173 EXPECT_TRUE(parser_->HandleControlFrameHeadersData(i, &(*it), 1)); | 188 EXPECT_TRUE(parser_->HandleControlFrameHeadersData(i, &(*it), 1)); |
174 EXPECT_EQ(SpdyHeadersBlockParser::NO_PARSER_ERROR, | 189 EXPECT_EQ(SpdyHeadersBlockParser::NO_PARSER_ERROR, |
175 parser_->get_error()); | 190 parser_->get_error()); |
176 } else { | 191 } else { |
177 EXPECT_FALSE(parser_->HandleControlFrameHeadersData(i, &(*it), 1)); | 192 EXPECT_FALSE(parser_->HandleControlFrameHeadersData(i, &(*it), 1)); |
178 EXPECT_EQ(SpdyHeadersBlockParser::NEED_MORE_DATA, parser_->get_error()); | 193 EXPECT_EQ(SpdyHeadersBlockParser::NEED_MORE_DATA, parser_->get_error()); |
179 } | 194 } |
180 } | 195 } |
181 } | 196 } |
182 } | 197 } |
183 | 198 |
184 TEST_P(SpdyHeadersBlockParserTest, HandlesEmptyCallsTest) { | 199 TEST_P(SpdyHeadersBlockParserTest, HandlesEmptyCallsTest) { |
185 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); | 200 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); |
186 | 201 |
187 string expect_key = kBaseKey + IntToString(0); | 202 string expect_key = kBaseKey + IntToString(0); |
188 string expect_value = kBaseValue + IntToString(0); | 203 string expect_value = kBaseValue + IntToString(0); |
189 EXPECT_CALL(handler_, OnHeader(StringPiece(expect_key), | 204 EXPECT_CALL(handler_, OnHeader(StringPiece(expect_key), |
190 StringPiece(expect_value))).Times(1); | 205 StringPiece(expect_value))).Times(1); |
191 | 206 |
192 string headers(CreateHeaders(1, false)); | 207 string headers(CreateHeaders(1, false)); |
193 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); | 208 if (FLAGS_chromium_http2_flag_log_compressed_size) { |
| 209 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length(), _)).Times(1); |
| 210 } else { |
| 211 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); |
| 212 } |
194 | 213 |
195 // Send a header in pieces with intermediate empty calls. | 214 // Send a header in pieces with intermediate empty calls. |
196 for (string::iterator it = headers.begin(); it != headers.end(); ++it) { | 215 for (string::iterator it = headers.begin(); it != headers.end(); ++it) { |
197 if ((it + 1) == headers.end()) { | 216 if ((it + 1) == headers.end()) { |
198 // Last byte completes the block. | 217 // Last byte completes the block. |
199 EXPECT_TRUE(parser_->HandleControlFrameHeadersData(1, &(*it), 1)); | 218 EXPECT_TRUE(parser_->HandleControlFrameHeadersData(1, &(*it), 1)); |
200 EXPECT_EQ(SpdyHeadersBlockParser::NO_PARSER_ERROR, parser_->get_error()); | 219 EXPECT_EQ(SpdyHeadersBlockParser::NO_PARSER_ERROR, parser_->get_error()); |
201 } else { | 220 } else { |
202 EXPECT_FALSE(parser_->HandleControlFrameHeadersData(1, &(*it), 1)); | 221 EXPECT_FALSE(parser_->HandleControlFrameHeadersData(1, &(*it), 1)); |
203 EXPECT_EQ(SpdyHeadersBlockParser::NEED_MORE_DATA, parser_->get_error()); | 222 EXPECT_EQ(SpdyHeadersBlockParser::NEED_MORE_DATA, parser_->get_error()); |
(...skipping 25 matching lines...) Expand all Loading... |
229 | 248 |
230 TEST_P(SpdyHeadersBlockParserTest, ExtraDataTest) { | 249 TEST_P(SpdyHeadersBlockParserTest, ExtraDataTest) { |
231 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); | 250 EXPECT_CALL(handler_, OnHeaderBlockStart()).Times(1); |
232 | 251 |
233 string expect_key = kBaseKey + IntToString(0); | 252 string expect_key = kBaseKey + IntToString(0); |
234 string expect_value = kBaseValue + IntToString(0); | 253 string expect_value = kBaseValue + IntToString(0); |
235 EXPECT_CALL(handler_, OnHeader(StringPiece(expect_key), | 254 EXPECT_CALL(handler_, OnHeader(StringPiece(expect_key), |
236 StringPiece(expect_value))).Times(1); | 255 StringPiece(expect_value))).Times(1); |
237 | 256 |
238 string headers = CreateHeaders(1, false) + "foobar"; | 257 string headers = CreateHeaders(1, false) + "foobar"; |
239 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); | 258 if (FLAGS_chromium_http2_flag_log_compressed_size) { |
| 259 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length(), _)).Times(1); |
| 260 } else { |
| 261 EXPECT_CALL(handler_, OnHeaderBlockEnd(headers.length())).Times(1); |
| 262 } |
240 | 263 |
241 EXPECT_FALSE(parser_->HandleControlFrameHeadersData(1, headers.c_str(), | 264 EXPECT_FALSE(parser_->HandleControlFrameHeadersData(1, headers.c_str(), |
242 headers.length())); | 265 headers.length())); |
243 EXPECT_EQ(SpdyHeadersBlockParser::TOO_MUCH_DATA, parser_->get_error()); | 266 EXPECT_EQ(SpdyHeadersBlockParser::TOO_MUCH_DATA, parser_->get_error()); |
244 } | 267 } |
245 | 268 |
246 TEST_P(SpdyHeadersBlockParserTest, WrongStreamIdTest) { | 269 TEST_P(SpdyHeadersBlockParserTest, WrongStreamIdTest) { |
247 string headers(CreateHeaders(kNumHeadersInBlock, false)); | 270 string headers(CreateHeaders(kNumHeadersInBlock, false)); |
248 EXPECT_FALSE(parser_->HandleControlFrameHeadersData(1, headers.data(), 1)); | 271 EXPECT_FALSE(parser_->HandleControlFrameHeadersData(1, headers.data(), 1)); |
249 EXPECT_EQ(SpdyHeadersBlockParser::NEED_MORE_DATA, parser_->get_error()); | 272 EXPECT_EQ(SpdyHeadersBlockParser::NEED_MORE_DATA, parser_->get_error()); |
250 bool result; | 273 bool result; |
251 EXPECT_SPDY_BUG( | 274 EXPECT_SPDY_BUG( |
252 result = parser_->HandleControlFrameHeadersData(2, headers.data() + 1, 1), | 275 result = parser_->HandleControlFrameHeadersData(2, headers.data() + 1, 1), |
253 "Unexpected stream id: 2 \\(expected 1\\)"); | 276 "Unexpected stream id: 2 \\(expected 1\\)"); |
254 EXPECT_FALSE(result); | 277 EXPECT_FALSE(result); |
255 EXPECT_EQ(SpdyHeadersBlockParser::UNEXPECTED_STREAM_ID, parser_->get_error()); | 278 EXPECT_EQ(SpdyHeadersBlockParser::UNEXPECTED_STREAM_ID, parser_->get_error()); |
256 } | 279 } |
257 | 280 |
258 TEST_P(SpdyHeadersBlockParserTest, InvalidStreamIdTest) { | 281 TEST_P(SpdyHeadersBlockParserTest, InvalidStreamIdTest) { |
259 string headers(CreateHeaders(kNumHeadersInBlock, false)); | 282 string headers(CreateHeaders(kNumHeadersInBlock, false)); |
260 bool result; | 283 bool result; |
261 EXPECT_SPDY_BUG( | 284 EXPECT_SPDY_BUG( |
262 result = parser_->HandleControlFrameHeadersData(0, headers.data(), 1), | 285 result = parser_->HandleControlFrameHeadersData(0, headers.data(), 1), |
263 "Expected nonzero stream id, saw: 0"); | 286 "Expected nonzero stream id, saw: 0"); |
264 EXPECT_FALSE(result); | 287 EXPECT_FALSE(result); |
265 EXPECT_EQ(SpdyHeadersBlockParser::UNEXPECTED_STREAM_ID, parser_->get_error()); | 288 EXPECT_EQ(SpdyHeadersBlockParser::UNEXPECTED_STREAM_ID, parser_->get_error()); |
266 } | 289 } |
267 | 290 |
268 } // namespace net | 291 } // namespace net |
OLD | NEW |