Summary: | Flaky Test: media/media-source/media-source-unnecessary-seek-seeked.html | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | WebKit Commit Bot <commit-queue> | ||||||
Component: | Media | Assignee: | WebKit Commit Bot <commit-queue> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eric.carlson, jer.noble, jlewis3, lforschler | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 50856 | ||||||||
Attachments: |
|
Description
WebKit Commit Bot
2017-06-13 17:18:49 PDT
Created attachment 312834 [details]
Archive of layout-test-results from webkit-cq-01
So the underlying cause of this failure is not new; but the recent change to delay firing the 'seeker' event until the frame at the seek destination is actually decoded makes this happen much more frequently. It goes like this: 1. MSE client enqueues a bunch of data to be parsed and enqueued 2. In the background, the media engine begins decoding the first visible frame 3. MSE client calls seek(), which enqueues a task to do the seek 4. The first frame is decoded, which triggers a readyState change. 5. The readyState change sees that the seek flag is set, and per the spec, finishes seeking 6. The seek task finally fires. Previous to this change, the ready state change in step 4 happens between step 2 and 3. After this change, it's much more likely that the ready state change in step 4 happens after the seek request in step 3. Created attachment 312859 [details]
Patch
*** Bug 173331 has been marked as a duplicate of this bug. *** Comment on attachment 312859 [details] Patch Clearing flags on attachment: 312859 Committed r218268: <http://trac.webkit.org/changeset/218268> All reviewed patches have been landed. Closing bug. |