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

Unified Diff: net/quic/quic_connection.cc

Issue 471613002: Change how QUIC negotiates pacing from congestion feedback to QUIC (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@Add_max_bandwidth_max_bandwidth_timestamp_73055131
Patch Set: Created 6 years, 4 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_connection.cc
diff --git a/net/quic/quic_connection.cc b/net/quic/quic_connection.cc
index c6f1a0ca718dcf3df8df434c4e3de2acaac19da7..a4305f62c71bfd1de98ed44f2e7e8cd6c1f5c288 100644
--- a/net/quic/quic_connection.cc
+++ b/net/quic/quic_connection.cc
@@ -237,10 +237,13 @@ QuicConnection::QuicConnection(QuicConnectionId connection_id,
peer_port_changed_(false),
self_ip_changed_(false),
self_port_changed_(false) {
+#if 0
+ // TODO(rtenneti): Should we enable this code in chromium?
ramant (doing other things) 2014/08/13 16:12:02 Hi Ryan and Ian, Do we need this code in chromiu
Ryan Hamilton 2014/08/13 16:33:40 Hm. Good question. I think we can probably leave t
ramant (doing other things) 2014/08/14 02:45:48 Acknowledged.
if (!is_server_) {
// Pacing will be enabled if the client negotiates it.
sent_packet_manager_.MaybeEnablePacing();
}
+#endif
DVLOG(1) << ENDPOINT << "Created connection with connection_id: "
<< connection_id;
timeout_alarm_->Set(clock_->ApproximateNow().Add(idle_network_timeout_));

Powered by Google App Engine
This is Rietveld 408576698