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/spdy/spdy_headers_block_parser_test.cc

Issue 822713002: Update from https://crrev.com/309415 (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 6 years 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/spdy/spdy_headers_block_parser_test.cc
diff --git a/net/spdy/spdy_headers_block_parser_test.cc b/net/spdy/spdy_headers_block_parser_test.cc
index dc2cbb5daecd20e38b178aa537017cb5a9135bb6..e2fdb311a1bbc268f6df018cbbca80486e52bb2a 100644
--- a/net/spdy/spdy_headers_block_parser_test.cc
+++ b/net/spdy/spdy_headers_block_parser_test.cc
@@ -39,6 +39,7 @@ class SpdyHeadersBlockParserTest :
void SetUp() override {
// Create a parser using the mock handler.
spdy_version_ = GetParam();
+
parser_.reset(new SpdyHeadersBlockParser(spdy_version_, &handler_));
length_field_size_ =
SpdyHeadersBlockParser::LengthFieldSizeForVersion(spdy_version_);
@@ -224,7 +225,7 @@ TEST_P(SpdyHeadersBlockParserTest, LargeBlocksDiscardedTest) {
EXPECT_EQ(SpdyHeadersBlockParser::HEADER_BLOCK_TOO_LARGE,
parser_->get_error());
}
- parser_->Reset();
+ parser_.reset(new SpdyHeadersBlockParser(spdy_version_, &handler_));
// Header block with one header, which has a too-long key.
{
EXPECT_CALL(handler_, OnHeaderBlock(1, 1)).Times(1);

Powered by Google App Engine
This is Rietveld 408576698