Land Recent QUIC Changes.
commite6ccc1a6314548b5cd74fc97513850e375bbcf92
authorrtenneti@chromium.org <rtenneti@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Mon, 2 Dec 2013 07:02:12 +0000 (2 07:02 +0000)
committerrtenneti@chromium.org <rtenneti@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Mon, 2 Dec 2013 07:02:12 +0000 (2 07:02 +0000)
tree964fda3a01d4a583e172bc56f12769b8078408cb
parentf79da8da073d7ee29619ac3d0e6f76ec0dc9c624
Land Recent QUIC Changes.

QUIC refactor to move the receive algorithm from QuicCongestionManager
into the ReceivedPacketManager.

Part of a larger congestion control refactor.

Merge internal change: 57325085
https://codereview.chromium.org/92593003/

Fix bug in cl/57315483 in which TerminateFromPeer(true) was turned into
CloseReadSide().  This looks correct, except that QuicSpdyServerStream
overrides this method so it wasn't being called.

Merge internal change: 57324551
https://codereview.chromium.org/93083002/

Rename QuicStreamSequencer::IsHalfClosed -> IsClosed. The sequencer only
has one path so it doesn't make much sense for it to be "half" closed.

Rename ReliableQuicStream::IsHalfClosed -> IsDoneReading
because the semantics are such that "IsHalfClosed" returned true when
the fin had been read, but before the read side had actually been
closed, which is confusing.

Rename two QUIC methods, no behavior change.

Merge internal change: 57315483
https://codereview.chromium.org/93073002/

The QuicAckNotifier should only be attached to a stream frame if the
stream frame contains data. If we didn't consume any data (e.g. it is a
FIN only frame), then attaching it to the frame results in
use-after-delete.

Fix use-after-delete in QuicAckNotifier.

Merge internal change: 57311555
https://codereview.chromium.org/92813003/

Removed the unused compressed parameter from CreateSynStream,
CreateSynReply and CreateHeaders methods.

Code cleanup, no behavioural change.

Merge internal change: 57308486
https://codereview.chromium.org/92793004/

Removing unnecessary received_truncated_ack_ local variable now that
acks have an explicit truncation flag.

Merge internal change: 57297471
https://codereview.chromium.org/92833003/

Change the Beta value used to reduce the TCP cubic congestion window in
QUIC to 1/12, the same as SPDY.

Merge internal change: 57269049
https://codereview.chromium.org/93033002/

R=rch@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238044 0039d316-1c4b-4281-b951-d872f2087c98
34 files changed:
net/quic/congestion_control/cubic.cc
net/quic/congestion_control/cubic_test.cc
net/quic/congestion_control/quic_congestion_manager.cc
net/quic/congestion_control/quic_congestion_manager.h
net/quic/congestion_control/receive_algorithm_interface.cc
net/quic/congestion_control/receive_algorithm_interface.h
net/quic/quic_ack_notifier_manager.cc
net/quic/quic_connection.cc
net/quic/quic_connection.h
net/quic/quic_connection_test.cc
net/quic/quic_packet_creator_test.cc
net/quic/quic_received_packet_manager.cc
net/quic/quic_received_packet_manager.h
net/quic/quic_received_packet_manager_test.cc
net/quic/quic_sent_packet_manager.cc
net/quic/quic_sent_packet_manager.h
net/quic/quic_sent_packet_manager_test.cc
net/quic/quic_spdy_compressor.cc
net/quic/quic_stream_sequencer.cc
net/quic/quic_stream_sequencer.h
net/quic/quic_stream_sequencer_test.cc
net/quic/reliable_quic_stream.cc
net/quic/reliable_quic_stream.h
net/quic/test_tools/quic_connection_peer.cc
net/quic/test_tools/quic_connection_peer.h
net/spdy/buffered_spdy_framer.cc
net/spdy/buffered_spdy_framer.h
net/spdy/buffered_spdy_framer_unittest.cc
net/spdy/spdy_framer.cc
net/spdy/spdy_framer.h
net/spdy/spdy_framer_test.cc
net/spdy/spdy_session.cc
net/spdy/spdy_test_util_common.cc
net/tools/flip_server/spdy_interface.cc