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

Unified Diff: net/quic/core/congestion_control/tcp_cubic_sender_bytes.cc

Issue 2550453002: Fixed integer signing bug in cubic bytes/packet implementation (Closed)
Patch Set: Created 4 years 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/core/congestion_control/tcp_cubic_sender_bytes.cc
diff --git a/net/quic/core/congestion_control/tcp_cubic_sender_bytes.cc b/net/quic/core/congestion_control/tcp_cubic_sender_bytes.cc
index f793211849784a7ef7a9142aeb7a71addbf60a7c..0f914b2847950114ad0d9384b6a37b4540e1712a 100644
--- a/net/quic/core/congestion_control/tcp_cubic_sender_bytes.cc
+++ b/net/quic/core/congestion_control/tcp_cubic_sender_bytes.cc
@@ -47,6 +47,16 @@ TcpCubicSenderBytes::TcpCubicSenderBytes(
TcpCubicSenderBytes::~TcpCubicSenderBytes() {}
+void TcpCubicSenderBytes::SetFromConfig(const QuicConfig& config,
+ Perspective perspective) {
+ TcpCubicSenderBase::SetFromConfig(config, perspective);
+ if (FLAGS_quic_fix_cubic_convex_mode &&
+ config.HasReceivedConnectionOptions() &&
+ ContainsQuicTag(config.ReceivedConnectionOptions(), kCCVX)) {
+ cubic_.SetFixConvexMode(true);
+ }
+}
+
void TcpCubicSenderBytes::SetCongestionWindowFromBandwidthAndRtt(
QuicBandwidth bandwidth,
QuicTime::Delta rtt) {

Powered by Google App Engine
This is Rietveld 408576698