SPDY Proxy - Bug fix to handle flow control (spdy/2.1)

net/spdy/spdy_http_stream.cc

 // Copyright (c) 2012 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_http_stream.h"
 
 #include <algorithm>
 #include <list>
 #include <string>
 
@@ skipping 111 matching lines @@
       } else {
         const int bytes_remaining = data->size() - bytes_to_copy;
         IOBufferWithSize* new_buffer = new IOBufferWithSize(bytes_remaining);
         memcpy(new_buffer->data(), &(data->data()[bytes_to_copy]),
                bytes_remaining);
         response_body_.pop_front();
         response_body_.push_front(make_scoped_refptr(new_buffer));
       }
       bytes_read += bytes_to_copy;
     }
-    if (spdy_session_ && spdy_session_->is_flow_control_enabled())
+    if (spdy_session_)

[willchan no longer on Chromium, 2012/02/14 19:16:44]

We don't need this if statement at all, do we?

[ramant (doing other things), 2012/02/16 00:23:15]

Done.

       stream_->IncreaseRecvWindowSize(bytes_read);
     return bytes_read;
   } else if (stream_->closed()) {
     return stream_->response_status();
   }
 
   CHECK(callback_.is_null());
   CHECK(!user_buffer_);
   CHECK_EQ(0, user_buffer_len_);
 
@@ skipping 349 matching lines @@
 bool SpdyHttpStream::IsSpdyHttpStream() const {
   return true;
 }
 
 void SpdyHttpStream::Drain(HttpNetworkSession* session) {
   Close(false);
   delete this;
 }
 
 }  // namespace net