Issue metadata
Sign in to add a comment
|
Fix regression in external/wpt/media-source/mediasource-append-buffer.html's "Test appendBuffer events order." |
||||||||||||||||||||||||
Issue descriptionRoll from WPT regressed this cas: https://chromium-review.googlesource.com/c/chromium/src/+/1107640
,
Jun 20 2018
Inspecting the contents of the roll, the test.step_func addition within it allows the function's assert failures to actually cause the test to fail. In Chrome before the roll, those asserts were sometimes failing, but the test object didn't know, nor report, such failures. So I'll look into whether the test or Chrome is behaving incorrectly.
,
Jun 20 2018
,
Jun 20 2018
The underlying issue exposed by this (IMHO correct test update in the roll) is a Chrome implementation problem: see bug 641121. tl;dr: readyState change to HAVE_METADATA and loadedmetadata event dispatch might not happen until *after* updateend event dispatch due to the way we plumb the readyState change across threads in Chrome currently. Fixing that will likely be easier once we do MSE parsing off the renderer main thread, to either eliminate the thread hop through pipeline thread, or to at least let the renderer proceed and not block while such hop is joined to ensure readyState transition in HTMLMediaElement is done on initial HAVE_METADATA prior to the updateend and !updating transition for the associated SourceBuffer having that last first init segment. workaround for web authors: wait for 'loadedmetadata' in such case; then readyState should be >= HAVE_METADATA.
,
Jun 20 2018
,
Jun 20 2018
,
Jun 20 2018
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by wolenetz@chromium.org
, Jun 20 2018