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

Unified Diff: net/quic/quic_sent_packet_manager.cc

Issue 1613513003: Early connection migration in QUIC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@home
Patch Set: Style nit fixed. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/quic_sent_packet_manager.cc
diff --git a/net/quic/quic_sent_packet_manager.cc b/net/quic/quic_sent_packet_manager.cc
index e6babd10638674cf04117532af80c1c745efb75c..8c9321af5b698500d3cb7ac492add10b1b04f4f1 100644
--- a/net/quic/quic_sent_packet_manager.cc
+++ b/net/quic/quic_sent_packet_manager.cc
@@ -36,6 +36,8 @@ static const int64_t kMaxRetransmissionTimeMs = 60000;
static const size_t kMaxRetransmissions = 10;
// Maximum number of packets retransmitted upon an RTO.
static const size_t kMaxRetransmissionsOnTimeout = 2;
+// Minimum number of consecutive RTOs before .
Ryan Hamilton 2016/02/02 04:19:59 Is this comment complete?
Jana 2016/02/02 06:53:53 Obviously not :-) I'm fixing this in the shared co
+const size_t kMinTimeoutsWhenPeerUnreachable = 2;
// Ensure the handshake timer isnt't faster than 10ms.
// This limits the tenth retransmitted packet to 10s after the initial CHLO.
@@ -1030,6 +1032,10 @@ bool QuicSentPacketManager::InSlowStart() const {
return send_algorithm_->InSlowStart();
}
+bool QuicSentPacketManager::IsPeerMaybeUnreachable() const {
+ return consecutive_rto_count_ >= kMinTimeoutsWhenPeerUnreachable;
+}
+
TransmissionInfo* QuicSentPacketManager::GetMutableTransmissionInfo(
QuicPacketNumber packet_number) {
return unacked_packets_.GetMutableTransmissionInfo(packet_number);

Powered by Google App Engine
This is Rietveld 408576698