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

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

Issue 2901773004: Landing Recent QUIC changes until May 20, 2017. (Closed)
Patch Set: Disable quic_restart_flag_quic_big_endian_connection_id_server until tests can be fixed. Created 3 years, 7 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/core/congestion_control/bbr_sender.cc
diff --git a/net/quic/core/congestion_control/bbr_sender.cc b/net/quic/core/congestion_control/bbr_sender.cc
index 5afad718ca0f3165f6af3cb1232c25274e8a7611..0d40c3fd76abb64de45a2508d388ac6bdf5e73c4 100644
--- a/net/quic/core/congestion_control/bbr_sender.cc
+++ b/net/quic/core/congestion_control/bbr_sender.cc
@@ -533,13 +533,10 @@ void BbrSender::UpdateRecoveryState(QuicPacketNumber last_acked_packet,
// Enter conservation on the first loss.
if (has_losses) {
recovery_state_ = CONSERVATION;
- if (FLAGS_quic_reloadable_flag_quic_bbr_fix_conservation ||
- FLAGS_quic_reloadable_flag_quic_bbr_fix_conservation2) {
+ if (FLAGS_quic_reloadable_flag_quic_bbr_fix_conservation2) {
// This will cause the |recovery_window_| to be set to the correct
// value in CalculateRecoveryWindow().
recovery_window_ = 0;
- QUIC_FLAG_COUNT_N(quic_reloadable_flag_quic_bbr_fix_conservation, 1,
- 3);
QUIC_FLAG_COUNT_N(quic_reloadable_flag_quic_bbr_fix_conservation2, 1,
3);
}
@@ -738,25 +735,10 @@ void BbrSender::CalculateRecoveryWindow(QuicByteCount bytes_acked,
switch (recovery_state_) {
case CONSERVATION:
- if (FLAGS_quic_reloadable_flag_quic_bbr_fix_conservation) {
- QUIC_FLAG_COUNT_N(quic_reloadable_flag_quic_bbr_fix_conservation, 2, 3);
- recovery_window_ =
- std::max(unacked_packets_->bytes_in_flight() + bytes_acked,
- recovery_window_);
- } else {
- recovery_window_ = unacked_packets_->bytes_in_flight() + bytes_acked;
- }
+ recovery_window_ = unacked_packets_->bytes_in_flight() + bytes_acked;
break;
case GROWTH:
- if (FLAGS_quic_reloadable_flag_quic_bbr_fix_conservation) {
- QUIC_FLAG_COUNT_N(quic_reloadable_flag_quic_bbr_fix_conservation, 3, 3);
- recovery_window_ =
- std::max(unacked_packets_->bytes_in_flight() + 2 * bytes_acked,
- recovery_window_ + bytes_acked);
- } else {
- recovery_window_ =
- unacked_packets_->bytes_in_flight() + 2 * bytes_acked;
- }
+ recovery_window_ = unacked_packets_->bytes_in_flight() + 2 * bytes_acked;
break;
default:
break;
« no previous file with comments | « net/quic/core/congestion_control/bandwidth_sampler_test.cc ('k') | net/quic/core/packet_number_indexed_queue.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698