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

Unified Diff: net/spdy/spdy_stream_test_util.cc

Issue 15701009: [SPDY] Add a SpdyStreamType enum and use it for SpdyStream (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: CHECK -> DCHECK Created 7 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
Index: net/spdy/spdy_stream_test_util.cc
diff --git a/net/spdy/spdy_stream_test_util.cc b/net/spdy/spdy_stream_test_util.cc
index fd1f14881d65084549d838986e4133e55a4174de..36c233ee1c00ea8f43d5cea994fc047099b2232f 100644
--- a/net/spdy/spdy_stream_test_util.cc
+++ b/net/spdy/spdy_stream_test_util.cc
@@ -20,9 +20,7 @@ ClosingDelegate::ClosingDelegate(
ClosingDelegate::~ClosingDelegate() {}
-SpdySendStatus ClosingDelegate::OnSendRequestHeadersComplete() {
- return NO_MORE_DATA_TO_SEND;
-}
+void ClosingDelegate::OnSendRequestHeadersComplete() {}
void ClosingDelegate::OnSendBody() {
ADD_FAILURE() << "OnSendBody should not be called";
@@ -58,11 +56,10 @@ StreamDelegateBase::StreamDelegateBase(
StreamDelegateBase::~StreamDelegateBase() {
}
-SpdySendStatus StreamDelegateBase::OnSendRequestHeadersComplete() {
+void StreamDelegateBase::OnSendRequestHeadersComplete() {
stream_id_ = stream_->stream_id();
EXPECT_NE(stream_id_, 0u);
send_headers_completed_ = true;
- return NO_MORE_DATA_TO_SEND;
}
int StreamDelegateBase::OnResponseReceived(const SpdyHeaderBlock& response,
@@ -164,11 +161,6 @@ StreamDelegateWithBody::StreamDelegateWithBody(
StreamDelegateWithBody::~StreamDelegateWithBody() {
}
-SpdySendStatus StreamDelegateWithBody::OnSendRequestHeadersComplete() {
- StreamDelegateBase::OnSendRequestHeadersComplete();
- return MORE_DATA_TO_SEND;
-}
-
void StreamDelegateWithBody::OnSendBody() {
stream()->SendStreamData(buf_.get(), buf_->size(), NO_MORE_DATA_TO_SEND);
}

Powered by Google App Engine
This is Rietveld 408576698