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

Unified Diff: net/spdy/spdy_proxy_client_socket.cc

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_proxy_client_socket.cc
diff --git a/net/spdy/spdy_proxy_client_socket.cc b/net/spdy/spdy_proxy_client_socket.cc
index 2c8ea6fc7bf83a940ea2a416281d8823b853b1b1..9ed7691b20d97e75d54d4b3bd4c3e5fe068e3fe9 100644
--- a/net/spdy/spdy_proxy_client_socket.cc
+++ b/net/spdy/spdy_proxy_client_socket.cc
@@ -441,10 +441,8 @@ void SpdyProxyClientSocket::OnRequestHeadersSent() {
OnIOComplete(OK);
}
-int SpdyProxyClientSocket::OnResponseHeadersReceived(
- const SpdyHeaderBlock& response,
- base::Time response_time,
- int status) {
+int SpdyProxyClientSocket::OnResponseHeadersUpdated(
+ const SpdyHeaderBlock& response_headers) {
// If we've already received the reply, existing headers are too late.
// TODO(mbelshe): figure out a way to make HEADERS frames useful after the
// initial response.
@@ -453,10 +451,10 @@ int SpdyProxyClientSocket::OnResponseHeadersReceived(
// Save the response
if (!SpdyHeadersToHttpResponse(
- response, spdy_stream_->GetProtocolVersion(), &response_))
+ response_headers, spdy_stream_->GetProtocolVersion(), &response_))
return ERR_INCOMPLETE_SPDY_HEADERS;
- OnIOComplete(status);
+ OnIOComplete(OK);
return OK;
}

Powered by Google App Engine
This is Rietveld 408576698