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

Issue 2242413002: Stop using next_packet_sequence_number_length in QuicPacketCreator. (Closed)

Created:
4 years, 4 months ago by ramant (doing other things)
Modified:
4 years, 4 months ago
Reviewers:
Ryan Hamilton
CC:
chromium-reviews, cbentzel+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@130098258
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Stop using next_packet_sequence_number_length in QuicPacketCreator. ENABLED FLAGS_quic_simple_packet_number_length_2. There's no functional change here. We rolled back FLAGS_quic_simple_packet_number_length because it didn't pick up the fix to b/29095779 cr/129995365 fixes that, so as that fix rolls out it's safe to turn FLAGS_quic_simple_packet_number_length on again. Merge internal change: 130113504 R=rch@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -27 lines) Patch
M net/quic/core/quic_connection.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M net/quic/core/quic_flags.h View 2 chunks +1 line, -1 line 0 comments Download
M net/quic/core/quic_flags.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M net/quic/core/quic_packet_creator.cc View 8 chunks +8 lines, -8 lines 0 comments Download
M net/quic/core/quic_packet_creator_test.cc View 12 chunks +12 lines, -12 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 2 (0 generated)
ramant (doing other things)
4 years, 4 months ago (2016-08-15 22:26:37 UTC) #1
Ryan Hamilton
4 years, 4 months ago (2016-08-15 23:25:14 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698