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

Unified Diff: net/quic/chromium/quic_chromium_client_stream_test.cc

Issue 2880643004: Move the async write handling from QuicChromiumClientStream to the Handle (Closed)
Patch Set: Rebase Created 3 years, 7 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/quic/chromium/quic_chromium_client_stream.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/chromium/quic_chromium_client_stream_test.cc
diff --git a/net/quic/chromium/quic_chromium_client_stream_test.cc b/net/quic/chromium/quic_chromium_client_stream_test.cc
index aba5b919929cc673918bb86112b4297c27c4b3a6..0623610f878860b1d4fed8e093a2e23bf693f2fb 100644
--- a/net/quic/chromium/quic_chromium_client_stream_test.cc
+++ b/net/quic/chromium/quic_chromium_client_stream_test.cc
@@ -641,7 +641,7 @@ TEST_P(QuicChromiumClientStreamTest, WriteStreamData) {
EXPECT_CALL(session_, WritevData(stream_, stream_->id(), _, _, _, _))
.WillOnce(Return(QuicConsumedData(kDataLen, true)));
TestCompletionCallback callback;
- EXPECT_EQ(OK, stream_->WriteStreamData(QuicStringPiece(kData1, kDataLen),
+ EXPECT_EQ(OK, handle_->WriteStreamData(QuicStringPiece(kData1, kDataLen),
true, callback.callback()));
}
@@ -657,7 +657,7 @@ TEST_P(QuicChromiumClientStreamTest, WriteStreamDataAsync) {
.WillOnce(Return(QuicConsumedData(0, false)));
TestCompletionCallback callback;
EXPECT_EQ(ERR_IO_PENDING,
- stream_->WriteStreamData(QuicStringPiece(kData1, kDataLen), true,
+ handle_->WriteStreamData(QuicStringPiece(kData1, kDataLen), true,
callback.callback()));
ASSERT_FALSE(callback.have_result());
@@ -682,7 +682,7 @@ TEST_P(QuicChromiumClientStreamTest, WritevStreamData) {
.WillOnce(Return(QuicConsumedData(buf2->size(), true)));
TestCompletionCallback callback;
EXPECT_EQ(
- OK, stream_->WritevStreamData({buf1, buf2}, {buf1->size(), buf2->size()},
+ OK, handle_->WritevStreamData({buf1, buf2}, {buf1->size(), buf2->size()},
true, callback.callback()));
}
@@ -702,7 +702,7 @@ TEST_P(QuicChromiumClientStreamTest, WritevStreamDataAsync) {
.WillOnce(Return(QuicConsumedData(0, false)));
TestCompletionCallback callback;
EXPECT_EQ(ERR_IO_PENDING,
- stream_->WritevStreamData({buf1.get(), buf2.get()},
+ handle_->WritevStreamData({buf1.get(), buf2.get()},
{buf1->size(), buf2->size()}, true,
callback.callback()));
ASSERT_FALSE(callback.have_result());
« no previous file with comments | « net/quic/chromium/quic_chromium_client_stream.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698