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

Unified Diff: net/spdy/spdy_stream_test_util.h

Issue 10828129: SPDY: WriteHeaders should not invoke OnDataSent callback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add check for OnHeadersSent in tests 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
Index: net/spdy/spdy_stream_test_util.h
diff --git a/net/spdy/spdy_stream_test_util.h b/net/spdy/spdy_stream_test_util.h
index 293bd7af39f78ea68430064f576b9c0c22fbef43..4e18919d62f30e1b9f8309c3164659cca5e0c25b 100644
--- a/net/spdy/spdy_stream_test_util.h
+++ b/net/spdy/spdy_stream_test_util.h
@@ -28,6 +28,7 @@ class TestSpdyStreamDelegate : public SpdyStream::Delegate {
virtual int OnResponseReceived(const SpdyHeaderBlock& response,
base::Time response_time,
int status) OVERRIDE;
+ virtual void OnHeadersSent() OVERRIDE;
virtual void OnDataReceived(const char* buffer, int bytes) OVERRIDE;
virtual void OnDataSent(int length) OVERRIDE;
virtual void OnClose(int status) OVERRIDE;
@@ -37,6 +38,7 @@ class TestSpdyStreamDelegate : public SpdyStream::Delegate {
return response_;
}
const std::string& received_data() const { return received_data_; }
+ int headers_sent() const { return headers_sent_; }
int data_sent() const { return data_sent_; }
bool closed() const { return closed_; }
@@ -48,6 +50,7 @@ class TestSpdyStreamDelegate : public SpdyStream::Delegate {
bool send_headers_completed_;
linked_ptr<SpdyHeaderBlock> response_;
std::string received_data_;
+ int headers_sent_;
int data_sent_;
bool closed_;

Powered by Google App Engine
This is Rietveld 408576698