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

Unified Diff: net/spdy/spdy_stream_unittest.cc

Issue 17382012: [SPDY] Refactor SpdyStream's handling of response headers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Forgot to rename a function Created 7 years, 6 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/spdy/spdy_stream_unittest.cc
diff --git a/net/spdy/spdy_stream_unittest.cc b/net/spdy/spdy_stream_unittest.cc
index df3c372c8b93cf1b5533f0759d3122ad3d8917f2..5bef6d79992302df6f004a2704ec6a2b64c04e90 100644
--- a/net/spdy/spdy_stream_unittest.cc
+++ b/net/spdy/spdy_stream_unittest.cc
@@ -228,22 +228,22 @@ TEST_P(SpdyStreamTest, PushedStream) {
kSpdyStreamInitialWindowSize,
net_log);
stream.set_stream_id(2);
- EXPECT_FALSE(stream.response_received());
+ EXPECT_FALSE(stream.ReceivedInitialResponseHeaders());
EXPECT_FALSE(stream.HasUrl());
// Set a couple of headers.
SpdyHeaderBlock response;
spdy_util_.AddUrlToHeaderBlock(kStreamUrl, &response);
- stream.OnResponseHeadersReceived(response);
+ stream.OnInitialResponseHeadersReceived(
+ response, base::Time::Now(), base::TimeTicks::Now());
// Send some basic headers.
SpdyHeaderBlock headers;
- response[spdy_util_.GetStatusKey()] = "200";
- response[spdy_util_.GetVersionKey()] = "OK";
- stream.OnHeaders(headers);
+ headers[spdy_util_.GetStatusKey()] = "200";
+ headers[spdy_util_.GetVersionKey()] = "OK";
+ stream.OnAdditionalResponseHeadersReceived(headers);
- stream.set_response_received();
- EXPECT_TRUE(stream.response_received());
+ EXPECT_TRUE(stream.ReceivedInitialResponseHeaders());
EXPECT_TRUE(stream.HasUrl());
EXPECT_EQ(kStreamUrl, stream.GetUrl().spec());

Powered by Google App Engine
This is Rietveld 408576698