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

Condition frame pacing on min/max playout delay

Browse Source 
As a first step we only want to enable frame pacing for the case
where min playout delay == 0 and max playout delay > 0.

Bug: chromium:1237402, chromium:1239469
Change-Id: Icf9641db7566083d0279135efa8618e435d881eb
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/228640
Commit-Queue: Johannes Kron <kron@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#34752}
This commit is contained in:
Johannes Kron
2021-08-13 10:24:04 +02:00
committed by WebRTC LUCI CQ
parent edfaaef086
commit b2745ba1f8
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
modules/video_coding/timing.cc
View File

@ -213,7 +213,8 @@ int64_t VCMTiming::MaxWaitingTime(int64_t render_time_ms,
bool too_many_frames_queued) const {
MutexLock lock(&mutex_);
if (render_time_ms == 0 && zero_playout_delay_min_pacing_->us() > 0) {
if (render_time_ms == 0 && zero_playout_delay_min_pacing_->us() > 0 &&
min_playout_delay_ms_ == 0 && max_playout_delay_ms_ > 0) {
// `render_time_ms` == 0 indicates that the frame should be decoded and
// rendered as soon as possible. However, the decoder can be choked if too
// many frames are sent at once. Therefore, limit the interframe delay to