Issue 2249783002: [m53 merge] Use LeastUnacked to calculate the packet number length instead of GetLeastPacketAwaited…

Issue 2249783002: [m53 merge] Use LeastUnacked to calculate the packet number length instead of GetLeastPacketAwaited… (Closed)

Created:
4 years, 4 months ago by Ryan Hamilton

Modified:
4 years, 4 months ago

Reviewers:
Zhongyi Shi

CC:
chromium-reviews, cbentzel+watch_chromium.org

Base URL:
https://chromium.googlesource.com/chromium/src.git@2785

Target Ref:
refs/pending/branch-heads/2785

Project:
chromium

Visibility:
Public.

[m53 merge] Use LeastUnacked to calculate the packet number length instead of GetLeastPacketAwaitedByPeer. Protected by default enabled FLAGS_quic_least_unacked_packet_number_length. Merge internal change: 128727157 Merge internal change: 129995365 BUG=636998 Committed: https://crrev.com/46da07b554cec9b784fbc08bc0899d9d39fc8d30 Cr-Commit-Position: refs/heads/master@{#411541} TBR=zhongyi@chromium.org, Committed: https://chromium.googlesource.com/chromium/src/+/f64bd662896f164cc9f4d3ea7eedf661cc0b9fb2

Created: 4 years, 4 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+28 lines, -6 lines)			Patch
M	net/quic/quic_connection.cc	View	2 chunks	+22 lines, -6 lines	0 comments	Download
M	net/quic/quic_flags.h	View	1 chunk	+1 line, -0 lines	0 comments	Download
M	net/quic/quic_flags.cc	View	1 chunk	+4 lines, -0 lines	0 comments	Download
M	net/quic/quic_sent_packet_manager.cc	View	1 chunk	+1 line, -0 lines	0 comments	Download

Total messages: 4 (3 generated)

Ryan Hamilton

4 years, 4 months ago (2016-08-15 19:51:17 UTC) #4

Message was sent while issue was closed.

Committed patchset #1 (id:1) manually as
f64bd662896f164cc9f4d3ea7eedf661cc0b9fb2 (presubmit successful).