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

Unified Diff: net/spdy/spdy_session_spdy2_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_proxy_client_socket.cc ('k') | net/spdy/spdy_session_spdy3_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_session_spdy2_unittest.cc
diff --git a/net/spdy/spdy_session_spdy2_unittest.cc b/net/spdy/spdy_session_spdy2_unittest.cc
index ecddf9b83efbfd76c2d7a07d76500ff7a8d46a12..15d78e070b1d837620a5409fe0fe2c627311be9d 100644
--- a/net/spdy/spdy_session_spdy2_unittest.cc
+++ b/net/spdy/spdy_session_spdy2_unittest.cc
@@ -43,6 +43,7 @@ class ClosingDelegate : public SpdyStream::Delegate {
int status) OVERRIDE {
return OK;
}
+ virtual void OnHeadersSent() OVERRIDE {}
virtual int OnDataReceived(const char* data, int length) OVERRIDE {
return OK;
}
@@ -74,30 +75,31 @@ class TestSpdyStreamDelegate : public net::SpdyStream::Delegate {
: callback_(callback) {}
virtual ~TestSpdyStreamDelegate() {}
- virtual bool OnSendHeadersComplete(int status) { return true; }
+ virtual bool OnSendHeadersComplete(int status) OVERRIDE { return true; }
- virtual int OnSendBody() {
+ virtual int OnSendBody() OVERRIDE {
return ERR_UNEXPECTED;
}
- virtual int OnSendBodyComplete(int /*status*/, bool* /*eof*/) {
+ virtual int OnSendBodyComplete(int /*status*/, bool* /*eof*/) OVERRIDE {
return ERR_UNEXPECTED;
}
virtual int OnResponseReceived(const SpdyHeaderBlock& response,
base::Time response_time,
- int status) {
+ int status) OVERRIDE {
return status;
}
- virtual int OnDataReceived(const char* buffer, int bytes) {
+ virtual void OnHeadersSent() OVERRIDE {}
+
+ virtual int OnDataReceived(const char* buffer, int bytes) OVERRIDE {
return OK;
}
- virtual void OnDataSent(int length) {
- }
+ virtual void OnDataSent(int length) OVERRIDE {}
- virtual void OnClose(int status) {
+ virtual void OnClose(int status) OVERRIDE {
CompletionCallback callback = callback_;
callback_.Reset();
callback.Run(OK);
« no previous file with comments | « net/spdy/spdy_proxy_client_socket.cc ('k') | net/spdy/spdy_session_spdy3_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698