Landing Recent QUIC changes until 10/11/2015 18:20 UTC

relnote:  Split QuicFecGroup interface into QuicFecGroupInterface.
Should cause no behavior change.

Implementing 'sliding window' FEC, part 1a.

Merge internal change: 105168235
https://codereview.chromium.org/1410593004/

relnote:  n/a (no changes to binary). Cleanup changes to tests, comments,
and formatting.

Merge internal change: 105084706
https://codereview.chromium.org/1418493004/

relnote: Reorganize how the read side of QUIC connections is closed down
when the consumer doesn't want any more data.  Flag protected by
FLAG_quic_implement_stop_reading; default true.

Remove CloseReadSide from the subclass interface provided by
ReliableQuicStream.  Subclasses (in particular QuicDataStream and
...QuicServerStream) now have an internal StopReading method which
causes all further incoming data to be ignored in the subclass code.
This prevents the stream object from being deleted until the stream is
properly closed.  It is necessary to keep the stream object functioning
until a FIN or RST_STREAM is received in order to send flow-control
window updates for the stream.  (When we define the RST_STREAM/NO_ERROR
mechanism to tell the peer to abort sending more data we will send
RST_STREAM/NO_ERROR as soon as this consumer is done writing as well.)

Re-install QuicSpdyServerStream::kErrorResponseBody as it is now used in
two places.

FIXED=21369841

Merge internal change: 105079850
https://codereview.chromium.org/1415603003/

relnote:  n/a (test change only). Call SetSpdyStreamCreator on the test
server before starting its thread, to avoid a data race.

Merge internal change: 104964040
https://codereview.chromium.org/1415783002/

relnote:  No functional change.  Flag protected by default-enabled
FLAGS_quic_fix_fin_accounting.

Revise processing of incoming stream frames for streams for which
reading is closed:  Record the FIN bit correctly.  This prevents
spurious entries in locally_closed_streams_highest_offset_[] and
consequent log messages "Surprisingly high number of locally closed
streams still waiting for final byte offset".

Investigating b/16010251.

Merge internal change: 104913996
https://codereview.chromium.org/1418553003/

relnote:  minor additions to quic-to-backend (not in production), change
fatal log message.

Add bidirectional data capability to the QUIC libraries.
Add accessors to the peer classes.

Merge internal change: 104332818
https://codereview.chromium.org/1417503004/

R=rch@chromium.org

Review URL: https://codereview.chromium.org/1414143002

Cr-Commit-Position: refs/heads/master@{#354994}

1 files changed, 2 insertions, 3 deletions

diff --git a/net/quic/quic_protocol.cc b/net/quic/quic_protocol.cc
index fedc16d..2038971 100644
--- a/net/quic/quic_protocol.cc
+++ b/net/quic/quic_protocol.cc
@@ -239,9 +239,8 @@ QuicAckFrame::QuicAckFrame()
 
 QuicAckFrame::~QuicAckFrame() {}
 
-QuicRstStreamErrorCode AdjustErrorForVersion(
-    QuicRstStreamErrorCode error_code,
-    QuicVersion version) {
+QuicRstStreamErrorCode AdjustErrorForVersion(QuicRstStreamErrorCode error_code,
+                                             QuicVersion /*version*/) {
   return error_code;
 }