commit e624d078bc5e11cd0577cd95c9d32b8880d368ec
author Erik Språng <sprang@webrtc.org> Wed Apr 11 16:47:27 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Apr 11 15:27:56 2018 +0000
tree 11627cf7183a5ca56d55631abd255b1e3cb11b05
parent e32bb5085521b59e62a7eb8476f86048ee6e0727

Always update libvpx configuration if bitrate has changed.

OnRatesUpdated() is called every time the bitrate estimate, or once per
second. However, since we don't want to reconfigure libvpx too often,
just in case it interferes with the rate controller, so
ScreenshareLayers contains a boolean |bitrate_update_| which indicate
if the configuration should be updated on a call to
UpdateConfiguration().

However, it two rate updates happened between two frames, the first of
which changes the rates and second one does not, |bitrate_update_| will
be reset to false and the encoder won't get the desired config.

This CL makes sure we update the configuration iff the rate has changed
at any time since the last call to UpdateConfiguration().

Bug: webrtc:9012
Change-Id: I62af36cffe20ecb7d3f403b3eb11f23a9692d719
Reviewed-on: https://webrtc-review.googlesource.com/69040
Commit-Queue: Erik Språng <sprang@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22826}