From 3ccda74d213ee5ee2313a926d5a88bae8c71c500 Mon Sep 17 00:00:00 2001 From: Sebastian Jansson Date: Tue, 24 Apr 2018 12:28:15 +0200 Subject: [PATCH] Removes unneeded DCHECK in BBR DataTransferTracker. If the packets are ordered by their ack time, their send time can decrease in the case they have been reordered. Bug: webrtc:8415 Change-Id: Ic9fca8d47de37b931085aeefcd62bbddd8869db9 Reviewed-on: https://webrtc-review.googlesource.com/72003 Commit-Queue: Sebastian Jansson Reviewed-by: Philip Eliasson Cr-Commit-Position: refs/heads/master@{#22999} --- modules/congestion_controller/bbr/data_transfer_tracker.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/modules/congestion_controller/bbr/data_transfer_tracker.cc b/modules/congestion_controller/bbr/data_transfer_tracker.cc index 5de85efb8b..44e5f1d4b6 100644 --- a/modules/congestion_controller/bbr/data_transfer_tracker.cc +++ b/modules/congestion_controller/bbr/data_transfer_tracker.cc @@ -25,7 +25,6 @@ void DataTransferTracker::AddSample(DataSize size_delta, Timestamp ack_time) { size_sum_ += size_delta; - RTC_DCHECK(samples_.empty() || send_time >= samples_.back().send_time); RTC_DCHECK(samples_.empty() || ack_time >= samples_.back().ack_time); if (!samples_.empty() && ack_time == samples_.back().ack_time) {