zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removes unneeded DCHECK in BBR DataTransferTracker.

Browse Source 
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 <srte@webrtc.org>
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22999}
This commit is contained in:
Sebastian Jansson
2018-04-24 12:28:15 +02:00
committed by Commit Bot
parent 29e865a5d8
commit 3ccda74d21
1 changed files with 0 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
modules/congestion_controller/bbr/data_transfer_tracker.cc
View File

@ -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) {