mirror of
https://chromium.googlesource.com/crosvm/crosvm
synced 2025-02-06 10:32:10 +00:00
06d7c9edb9
CL crrev.com/c/3005194 introduced changes to the encoder to allow dynamic framerate changes even when there are already resources queued in the input or output queue. As a side effect the encoder is only recreated if any parameters besides the framerate change. Unfortunately the above checks assumes that changing the framerate will never influence any of the other parameters, which isn't always correct. The requested H.264 level is adjusted to conform to the minimum requirements for the selected bitrate and framerate. We can't dynamically adjust the H.264 level during encoding, but as long as we don't have any resources queued yet we can recreate the encoder to make sure the H.264 level is properly adjusted if required. BUG=b:192623395,b:192419592,b:193947502 TEST=android.media.cts.MediaRecorderTest#testProfileAvcBaselineLevel1 Change-Id: Ie02e30a5b6fb82b9b62e88ae3ab75c8ef42f3844 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3037296 Tested-by: kokoro <noreply+kokoro@google.com> Auto-Submit: David Staessens <dstaessens@chromium.org> Commit-Queue: David Staessens <dstaessens@chromium.org> Reviewed-by: Alexandre Courbot <acourbot@chromium.org> |
||
---|---|---|
.. | ||
src | ||
.build_test_serial | ||
Cargo.toml |