Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(83)

Issue 1424653008: Make QUIC stateless rejects work in the face of packet loss by storing the SREJ packet(s) in the Qu… (Closed)

Created:
5 years, 1 month ago by Ryan Hamilton
Modified:
5 years, 1 month ago
CC:
chromium-reviews, cbentzel+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@106432163
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Make QUIC stateless rejects work in the face of packet loss by storing the SREJ packet(s) in the QuicTimeWaitListManager so they can be retransmitted. Store QUIC SREJ packet(s) in the QuicTimeWaitListManager. Merge internal change: 106433678 R=rtennet@chromium.org BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+164 lines, -90 lines) Patch
M net/quic/quic_connection.h View 4 chunks +13 lines, -6 lines 0 comments Download
M net/quic/quic_connection.cc View 7 chunks +23 lines, -8 lines 0 comments Download
M net/quic/quic_crypto_server_stream.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M net/quic/test_tools/quic_connection_peer.cc View 1 chunk +5 lines, -1 line 0 comments Download
M net/tools/quic/end_to_end_test.cc View 2 chunks +7 lines, -16 lines 0 comments Download
M net/tools/quic/quic_dispatcher.cc View 1 chunk +6 lines, -4 lines 0 comments Download
M net/tools/quic/quic_time_wait_list_manager.h View 2 chunks +18 lines, -25 lines 0 comments Download
M net/tools/quic/quic_time_wait_list_manager.cc View 5 chunks +37 lines, -19 lines 0 comments Download
M net/tools/quic/quic_time_wait_list_manager_test.cc View 5 chunks +44 lines, -8 lines 0 comments Download
M net/tools/quic/test_tools/mock_quic_time_wait_list_manager.h View 1 chunk +4 lines, -3 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 4 (2 generated)
Ryan Hamilton
5 years, 1 month ago (2015-11-02 20:37:09 UTC) #1
ramant (doing other things)
5 years, 1 month ago (2015-11-03 00:35:32 UTC) #4
lgtm

Powered by Google App Engine
This is Rietveld 408576698