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

Unified Diff: net/spdy/spdy_websocket_stream.h

Issue 17382012: [SPDY] Refactor SpdyStream's handling of response headers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Forgot to rename a function Created 7 years, 6 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_websocket_stream.h
diff --git a/net/spdy/spdy_websocket_stream.h b/net/spdy/spdy_websocket_stream.h
index 903c0bce311368b546fdc47486fd343ee5297925..fccf4bb74dd61e91dc9fbd52907560824750d92d 100644
--- a/net/spdy/spdy_websocket_stream.h
+++ b/net/spdy/spdy_websocket_stream.h
@@ -38,13 +38,12 @@ class NET_EXPORT_PRIVATE SpdyWebSocketStream
// has been sent.
virtual void OnSentSpdyHeaders() = 0;
- // Called on corresponding to OnResponseHeadersReceived() or
+ // Called on corresponding to OnResponseHeadersUpdated() or
// SPDY's SYN_STREAM, SYN_REPLY, or HEADERS frames are
// received. This callback may be called multiple times as SPDY's
// delegate does.
- virtual int OnReceivedSpdyResponseHeader(
- const SpdyHeaderBlock& headers,
- int status) = 0;
+ virtual void OnSpdyResponseHeadersUpdated(
+ const SpdyHeaderBlock& response_headers) = 0;
// Called when data is sent.
virtual void OnSentSpdyData(size_t bytes_sent) = 0;
@@ -76,9 +75,8 @@ class NET_EXPORT_PRIVATE SpdyWebSocketStream
// SpdyStream::Delegate
virtual void OnRequestHeadersSent() OVERRIDE;
- virtual int OnResponseHeadersReceived(const SpdyHeaderBlock& response,
- base::Time response_time,
- int status) OVERRIDE;
+ virtual int OnResponseHeadersUpdated(
+ const SpdyHeaderBlock& response_headers) OVERRIDE;
virtual int OnDataReceived(scoped_ptr<SpdyBuffer> buffer) OVERRIDE;
virtual void OnDataSent() OVERRIDE;
virtual void OnClose(int status) OVERRIDE;

Powered by Google App Engine
This is Rietveld 408576698