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

Unified Diff: net/spdy/spdy_stream_spdy3_unittest.cc

Issue 10828129: SPDY: WriteHeaders should not invoke OnDataSent callback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 4 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
« no previous file with comments | « net/spdy/spdy_stream_spdy2_unittest.cc ('k') | net/spdy/spdy_stream_test_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_stream_spdy3_unittest.cc
diff --git a/net/spdy/spdy_stream_spdy3_unittest.cc b/net/spdy/spdy_stream_spdy3_unittest.cc
index 3a3ad6620e134a215d4d7099ef19ee5b0adf222c..313756a242c835407f23b02632e6bd1349439b66 100644
--- a/net/spdy/spdy_stream_spdy3_unittest.cc
+++ b/net/spdy/spdy_stream_spdy3_unittest.cc
@@ -267,10 +267,9 @@ TEST_F(SpdyStreamSpdy3Test, SendHeaderAndDataAfterOpen) {
EXPECT_TRUE(delegate->send_headers_completed());
EXPECT_EQ("101", (*delegate->response())[":status"]);
+ EXPECT_EQ(1, delegate->headers_sent());
EXPECT_EQ(std::string(), delegate->received_data());
- // TODO(toyoshim): OnDataSent should be invoked when each data frame is sent.
- // But current implementation invokes also when each HEADERS frame is sent.
- //EXPECT_EQ(6, delegate->data_sent());
+ EXPECT_EQ(6, delegate->data_sent());
}
TEST_F(SpdyStreamSpdy3Test, PushedStream) {
« no previous file with comments | « net/spdy/spdy_stream_spdy2_unittest.cc ('k') | net/spdy/spdy_stream_test_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698