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

Issue 1435553004: Add OnConnectionMigration method to QuicSentPacketManager. Add OnConnectionMigration interface to S… (Closed)

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

Description

Add OnConnectionMigration method to QuicSentPacketManager. Add OnConnectionMigration interface to SendAlgorithmInterface and RttStats. No functional change expected. This change is preparing to reset cwnd and rtt measurement when the client migrates to a different network. Merge internal change: 106851267 R=rch@chromium.org BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -0 lines) Patch
M net/quic/congestion_control/pacing_sender.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/quic/congestion_control/rtt_stats.h View 1 chunk +3 lines, -0 lines 0 comments Download
M net/quic/congestion_control/send_algorithm_interface.h View 1 chunk +3 lines, -0 lines 0 comments Download
M net/quic/congestion_control/tcp_cubic_bytes_sender.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/quic/congestion_control/tcp_cubic_sender.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/quic/quic_connection.cc View 1 chunk +1 line, -0 lines 0 comments Download
M net/quic/quic_sent_packet_manager.h View 1 chunk +3 lines, -0 lines 0 comments Download
M net/quic/quic_sent_packet_manager.cc View 1 chunk +15 lines, -0 lines 0 comments Download
M net/quic/test_tools/quic_test_utils.h View 1 chunk +1 line, -0 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 2 (0 generated)
rjshade
5 years, 1 month ago (2015-11-10 15:12:19 UTC) #1
Ryan Hamilton
5 years, 1 month ago (2015-11-10 21:38:05 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698