relnote: Change the QuicStreamSequencer to call OnDataAvailable instead

net/quic/quic_reliable_client_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/quic/quic_reliable_client_stream.h"
 
 #include "base/callback_helpers.h"
 #include "base/location.h"
 #include "base/thread_task_runner_handle.h"
 #include "net/base/io_buffer.h"
@@ skipping 59 matching lines @@
   }
 }
 
 QuicPriority QuicReliableClientStream::EffectivePriority() const {
   if (delegate_ && delegate_->HasSendHeadersComplete()) {
     return QuicDataStream::EffectivePriority();
   }
   return QuicWriteBlockedList::kHighestPriority;
 }
 
-void QuicReliableClientStream::OnFinRead() {
-  // Do not close the stream here to give the caller a chance to
-  // invoke DoneReading.
-}
-
 void QuicReliableClientStream::DoneReading() {

[Ryan Hamilton, 2015/08/04 20:28:56]

Might as well get right of this method, and change callers of DoneReading() to
call OnFinRead() instead. What do you think?

[ramant (doing other things), 2015/08/05 23:35:16]

Done.

   QuicDataStream::OnFinRead();
 }
 
 int QuicReliableClientStream::WriteStreamData(
     base::StringPiece data,
     bool fin,
     const CompletionCallback& callback) {
   // We should not have data buffered.
   DCHECK(!HasBufferedData());
   // Writes the data, or buffers it.
@@ skipping 81 matching lines @@
       base::Bind(&QuicReliableClientStream::NotifyDelegateOfDataAvailable,
                  weak_factory_.GetWeakPtr()));
 }
 
 void QuicReliableClientStream::NotifyDelegateOfDataAvailable() {
   if (delegate_)
     delegate_->OnDataAvailable();
 }
 
 }  // namespace net