Add chunked uploads support to SPDY

net/spdy/spdy_proxy_client_socket.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/spdy/spdy_proxy_client_socket.h"
 
 #include <algorithm>  // min
 
 #include "base/logging.h"
 #include "base/string_util.h"
@@ skipping 386 matching lines @@
   return true;
 }
 
 int SpdyProxyClientSocket::OnSendBody() {
   // Because we use |spdy_stream_| via STATE_OPEN (ala WebSockets)
   // OnSendBody() should never be called.
   NOTREACHED();
   return ERR_UNEXPECTED;
 }
 
-bool SpdyProxyClientSocket::OnSendBodyComplete(int status) {
+int SpdyProxyClientSocket::OnSendBodyComplete(int /*status*/, bool* /*eof*/) {
   // Because we use |spdy_stream_| via STATE_OPEN (ala WebSockets)
   // OnSendBodyComplete() should never be called.
   NOTREACHED();
-  return false;
+  return ERR_UNEXPECTED;
 }
 
 int SpdyProxyClientSocket::OnResponseReceived(
     const spdy::SpdyHeaderBlock& response,
     base::Time response_time,
     int status) {
   // 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.
   if (next_state_ != STATE_READ_REPLY_COMPLETE)
@@ skipping 69 matching lines @@
     read_callback_ = NULL;
     read_callback->Run(status);
   } else if (read_callback_) {
     // If we have a read_callback, the we need to make sure we call it back
     OnDataReceived(NULL, 0);
   }
   if (write_callback)
     write_callback->Run(ERR_CONNECTION_CLOSED);
 }
 
+void SpdyProxyClientSocket::set_chunk_callback(ChunkCallback* /*callback*/) {
+  NOTREACHED();

[Satish, 2011/03/03 23:57:06]

Looks like adding this was incorrect as SpdyStream is calling this method when
sending a request and detaching. SpdyStream doesn't know about the actual POST
data so it can't call this method only for chunked POST data. Is it ok if I
remove the check and leave the method empty?

[willchan no longer on Chromium, 2011/03/04 00:02:55]

OK

+}
+
 }  // namespace net