Chromium Code Reviews Chromium Code Reviews
Issue
447093004:
Land Recent QUIC Changes. (Closed)
DescriptionLand Recent QUIC Changes.
QUIC: don't keep around state for streams created by a request
w/no body. Waiting for a stream frame w/ FIN that will never arrive, and
the closed stream map grows without bound. Not flag protected.
Merge internal change: 72338396
https://codereview.chromium.org/447093002/
Default socket receive buffer to 256K bytes and remove unnecessary
CongestionFeedbackFrames from tests.
Merge internal change: 72338299
https://codereview.chromium.org/447083002/
Include the socket receive buffer in the connection handshake.
PRESENCE_OPTIONAL
The goal is to eventually remove the TcpCongestionFeedbackFrame.
Currently this sends the same value for every packet. All it contains is
the tcp_receive_window (now called socket_receive_buffer) which is fixed
at 256000. So it can be sent in the connection establishment and then
never retransmitted.
Because only one type of CongestionFeedbackFrame can be sent with an
ack, this frees up space for a TimestampFrame which is the motivation
for removing the TcpCongestionFeedbackFrame.
Merge internal change: 72334240
https://codereview.chromium.org/446283002/
Improve QUIC's SendAlgorithmSimulator to allow setting the delayed ack
timer and add a new test of a second BBR flow starting after a first
flow has stabilized.
Merge internal change: 72332690
https://codereview.chromium.org/449713002/
QUIC congestion option that forces ICWND to 10. Not flag protected.
Merge internal change: 72323017
https://codereview.chromium.org/446253002/
Allow receiving a QUIC CongestionFeedbackFrame that is not kTCP.
Currently the only congestion feedback frames that are sent are TCP, but
I want to send timestamp packets in the next version, so the send
algorithms should not assume that the incoming congestion feedback frame
is kTCP.
Merge internal change: 72322097
https://codereview.chromium.org/443313002/
QUIC - minor cleanup to match internal source tree.
* Formatting cleanup and .get() != NULL changes.
Merge internal change: 72204223
https://codereview.chromium.org/447973004/
Bring back InterArrivalReceiver and rename it to TimestampeReceiver.
This is not in production.
Merge internal change: 72170964
https://codereview.chromium.org/449693002/
Move Quic AppendTimestampFrame method out of AppendCongestionFrame.
Also, remove obsolete TODO's and rename InterArrival to Timestamp.
Not used in production.
Merge internal change: 72148809
https://codereview.chromium.org/446063005/
R=rch@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288247
Patch Set 1 #Patch Set 2 : fixed indenation in QuicFramer.cc #Messages
Total messages: 8 (0 generated)
|
