We hit the assertion in patch2 when running the crash test 1389304.html [1], because the first demuxed video frame of the video in that test is not the first frame for presentation, which causes the result of adjustment being negative. Therefore, we have to adjust our start time manually after we get the first decoded sample from both audio and video. Will ask for a review if there is no any problem on the try result [2]. [1] https://searchfox.org/mozilla-central/source/dom/media/test/crashtests/1389304.html [2] https://treeherder.mozilla.org/#/jobs?repo=try&revision=7345a1b646ad0aa564c6979dee82bb3ea317920a
Bug 1540748 Comment 19 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
We hit the assertion in patch2 when running the crash test 1389304.html [1], because the first demuxed video frame of the video in that test is not the first frame for presentation, which causes the result of adjustment being negative. Therefore, we have to adjust our start time manually after we get the first decoded sample from both audio and video. Will ask for a review if there is no any problem on the try result [2]. [1] https://searchfox.org/mozilla-central/source/dom/media/test/crashtests/1389304.html [2] https://treeherder.mozilla.org/#/jobs?repo=try&revision=a51efb3c7bde8102944da0877a293772f2f6697e