http/tests/media/video-play-stall.html has started to time out every time on macOS Sierra and on Gtk after https://trac.webkit.org/r220052 (Remove code in HTMLObjectElement attribute parsing that forces style resolution and layout). Happens with both WK1 and WK2. It passes on El Capitan, which is why EWS didn't catch this.
Should be easy to fix. I fixed lots of other bugs like this one.
It would be good to skip this test to make the bots green. I promise I will re-enable it when I fix it.
Skipped in: https://trac.webkit.org/changeset/220075/webkitx
(In reply to Matt Lewis from comment #3) > Skipped in: > https://trac.webkit.org/changeset/220075/webkitx typo https://trac.webkit.org/changeset/220075/webkit
Created attachment 316836 [details] Patch
In fixing this bug I learned that it's very hard to get HTMLMediaElement to ever cancel a load once it's going. It even keeps loading when you switch src to load a different video. That is not a good thing, and worth investigating and fixing separately. It’s also not great that starting a load from inside a load event handler has any effect on the loading state at all. By the time the load event has been sent, it should be too late for new loads to have this kind of effect. We should make some other tests for that while making future fixes to the frame loading logic.
I’d love to get this test re-enabled. Anyone willing to review?
Comment on attachment 316836 [details] Patch Clearing flags on attachment: 316836 Committed r220098: <http://trac.webkit.org/changeset/220098>
All reviewed patches have been landed. Closing bug.
<rdar://problem/33655124>