gfe-relnote: (n/a) Deprecate --gfe2_reloadable_flag_quic_always_get_bw_sample_when_acked. PiperOrigin-RevId: 250718928 Change-Id: I77aef64913cb789cbdf165551ba2818c8efb6adf
diff --git a/quic/core/congestion_control/bbr_sender.cc b/quic/core/congestion_control/bbr_sender.cc index 5844987..3c975df 100644 --- a/quic/core/congestion_control/bbr_sender.cc +++ b/quic/core/congestion_control/bbr_sender.cc
@@ -137,9 +137,7 @@ probe_rtt_skipped_if_similar_rtt_(false), probe_rtt_disabled_if_app_limited_(false), app_limited_since_last_probe_rtt_(false), - min_rtt_since_last_probe_rtt_(QuicTime::Delta::Infinite()), - always_get_bw_sample_when_acked_( - GetQuicReloadableFlag(quic_always_get_bw_sample_when_acked)) { + min_rtt_since_last_probe_rtt_(QuicTime::Delta::Infinite()) { if (stats_) { stats_->slowstart_count = 0; stats_->slowstart_start_time = QuicTime::Zero(); @@ -526,14 +524,9 @@ const AckedPacketVector& acked_packets) { QuicTime::Delta sample_min_rtt = QuicTime::Delta::Infinite(); for (const auto& packet : acked_packets) { - if (!always_get_bw_sample_when_acked_ && packet.bytes_acked == 0) { - // Skip acked packets with 0 in flight bytes when updating bandwidth. - continue; - } BandwidthSample bandwidth_sample = sampler_.OnPacketAcknowledged(now, packet.packet_number); - if (always_get_bw_sample_when_acked_ && - !bandwidth_sample.state_at_send.is_valid) { + if (!bandwidth_sample.state_at_send.is_valid) { // From the sampler's perspective, the packet has never been sent, or the // packet has been acked or marked as lost previously. continue;
diff --git a/quic/core/congestion_control/bbr_sender.h b/quic/core/congestion_control/bbr_sender.h index a49a5ef..b559134 100644 --- a/quic/core/congestion_control/bbr_sender.h +++ b/quic/core/congestion_control/bbr_sender.h
@@ -402,9 +402,6 @@ bool probe_rtt_disabled_if_app_limited_; bool app_limited_since_last_probe_rtt_; QuicTime::Delta min_rtt_since_last_probe_rtt_; - - // Latched value of --quic_always_get_bw_sample_when_acked. - const bool always_get_bw_sample_when_acked_; }; QUIC_EXPORT_PRIVATE std::ostream& operator<<(std::ostream& os,