-
Notifications
You must be signed in to change notification settings - Fork 30
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge to M81: WebmMuxer: Do not produce duplicate timestamps.
This change fixes a problem where the WebmMuxer will produce duplicate timestamps leading to audio gaps or jerky video in WebM files, provided both audio and video is recorded, and timestamps of all incoming frames are not monotonically increasing. This resulted in the WebmMuxer rewriting apparently older frames' timestamps as the newest one encountered. With this change we now normally don't change the incoming timestamps, but always output a sorted sequence of frames to the underlying Matroska muxer. Timestamps are exceptionally changed when it's found that each stream's timestamps aren't monotonically increasing. Additionally, the webm_muxer got it's unit tests updated. TBR=mcasas@chromium.org (cherry picked from commit 4234d1e) Bug: 873963 Change-Id: I6b81ef32f0f7f1034a56da3bca07186221fefaea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2033253 Commit-Queue: Markus Handell <handellm@google.com> Reviewed-by: Miguel Casas <mcasas@chromium.org> Reviewed-by: Dale Curtis <dalecurtis@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#738738} Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2050369 Reviewed-by: Markus Handell <handellm@google.com> Cr-Commit-Position: refs/branch-heads/4044@{#197} Cr-Branched-From: a6d9daf-refs/heads/master@{#737173}
- Loading branch information
Showing
3 changed files
with
210 additions
and
157 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.