From 7b6acd68ee44c046e922291af380065002929482 Mon Sep 17 00:00:00 2001 From: Christoffer Rodbro Date: Wed, 27 Mar 2019 12:52:33 +0000 Subject: [PATCH] Revert "Avoid calling OnRoundTripTimeUpdate with invalid RTTs." This reverts commit afa61c94e50e2737d4d4b22d7a830845e763cf27. Reason for revert: Breaks a downstream test. Original change's description: > Avoid calling OnRoundTripTimeUpdate with invalid RTTs. > > Bug: none > Change-Id: Ic19b87ad7094465da6091d0e99b10a6d1b7d2e58 > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/128776 > Commit-Queue: Christoffer Rodbro > Reviewed-by: Sebastian Jansson > Cr-Commit-Position: refs/heads/master@{#27308} TBR=srte@webrtc.org,crodbro@webrtc.org Change-Id: Ic4c516d3325050858ac99731f6d25181fb40b7bd No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: none Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/129922 Reviewed-by: Christoffer Rodbro Commit-Queue: Christoffer Rodbro Cr-Commit-Position: refs/heads/master@{#27309} --- call/rtp_transport_controller_send.cc | 2 +- .../congestion_controller/goog_cc/goog_cc_network_control.cc | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/call/rtp_transport_controller_send.cc b/call/rtp_transport_controller_send.cc index ab95907ac8..a414c3b425 100644 --- a/call/rtp_transport_controller_send.cc +++ b/call/rtp_transport_controller_send.cc @@ -400,7 +400,7 @@ void RtpTransportControllerSend::OnReceivedRtcpReceiverReport( report.receive_time = Timestamp::ms(now_ms); report.round_trip_time = TimeDelta::ms(rtt_ms); report.smoothed = false; - if (controller_ && !report.round_trip_time.IsZero()) + if (controller_) PostUpdates(controller_->OnRoundTripTimeUpdate(report)); }); } diff --git a/modules/congestion_controller/goog_cc/goog_cc_network_control.cc b/modules/congestion_controller/goog_cc/goog_cc_network_control.cc index e39d43f615..283ca983e9 100644 --- a/modules/congestion_controller/goog_cc/goog_cc_network_control.cc +++ b/modules/congestion_controller/goog_cc/goog_cc_network_control.cc @@ -229,7 +229,6 @@ NetworkControlUpdate GoogCcNetworkController::OnRoundTripTimeUpdate( RoundTripTimeUpdate msg) { if (packet_feedback_only_ || msg.smoothed) return NetworkControlUpdate(); - RTC_DCHECK(!msg.round_trip_time.IsZero()); if (delay_based_bwe_) delay_based_bwe_->OnRttUpdate(msg.round_trip_time); bandwidth_estimation_->UpdateRtt(msg.round_trip_time, msg.receive_time);