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

Removes timeout warnings from pacer before first packet.

Browse Source 
The paced sender did not update the time out clock before the first
packet was send in paused state. This caused it to incorrectly log
warnings about elapsed time. This CL fixes this.

Bug: None
Change-Id: I240d169464a708c12eb580d57bc385330b8dd6b1
Reviewed-on: https://webrtc-review.googlesource.com/69561
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22852}
This commit is contained in:
Sebastian Jansson
2018-04-12 13:08:22 +02:00
committed by Commit Bot
parent 70488de694
commit 682c6498a2
1 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
modules/pacing/paced_sender.cc
View File

@ -268,12 +268,14 @@ void PacedSender::Process() {
// TODO(srte): Stop sending packet in paused state when pause is no longer
// used for congestion windows.
if (paused_ || Congested()) {
// We can not send padding unless a normal packet has first been sent. If we
// do, timestamps get messed up.
if (elapsed_time_ms >= kCongestedPacketIntervalMs && packet_counter_ > 0) {
PacedPacketInfo pacing_info;
size_t bytes_sent = SendPadding(1, pacing_info);
alr_detector_->OnBytesSent(bytes_sent, elapsed_time_ms);
if (elapsed_time_ms >= kCongestedPacketIntervalMs) {
// We can not send padding unless a normal packet has first been sent. If
// we do, timestamps get messed up.
if (packet_counter_ > 0) {
PacedPacketInfo pacing_info;
size_t bytes_sent = SendPadding(1, pacing_info);
alr_detector_->OnBytesSent(bytes_sent, elapsed_time_ms);
}
last_send_time_us_ = clock_->TimeInMicroseconds();
}
return;