Don't signal updated bitrate allocation on encoder paused
If the paced queue gets too long becaused of e.g. encoder overshoot,
the encoder is paused by setting the target bitrate to 0. Don't signal
this 0-bitrate via RTCP TargetBitrate message as the overall target
bitrate is probably unchanged.
Bug: webrtc:9734
Change-Id: I77f23b707a8d4494d0c89fa05005ac1482eace52
Reviewed-on: https://webrtc-review.googlesource.com/99507
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Erik Språng <sprang@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24679}diff --git a/video/video_send_stream_impl.cc b/video/video_send_stream_impl.cc
index aa9e6c9..3241b0a 100644
--- a/video/video_send_stream_impl.cc
+++ b/video/video_send_stream_impl.cc
@@ -432,7 +432,10 @@
void VideoSendStreamImpl::OnBitrateAllocationUpdated(
const VideoBitrateAllocation& allocation) {
- rtp_video_sender_->OnBitrateAllocationUpdated(allocation);
+ if (encoder_target_rate_bps_ != 0) {
+ // Send bitrate allocation metadata only if encoder is not paused.
+ rtp_video_sender_->OnBitrateAllocationUpdated(allocation);
+ }
}
void VideoSendStreamImpl::SignalEncoderActive() {
