RESOLVED FIXED 147045
Video posters disappear once media has loaded 
https://bugs.webkit.org/show_bug.cgi?id=147045
Summary Video posters disappear once media has loaded
Myles C. Maxfield
Reported 2015-07-17 13:22:40 PDT
Video posters disappear once media has loaded
Attachments
Patch (3.85 KB, patch)
2015-07-17 13:26 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (4.32 KB, patch)
2015-07-17 14:37 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (4.39 KB, patch)
2015-07-17 15:16 PDT, Myles C. Maxfield 		simon.fraser: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2015-07-17 13:26:04 PDT
Created attachment 256982 [details] Patch
Myles C. Maxfield
Comment 2 2015-07-17 13:32:41 PDT
<rdar://problem/21359811>
Simon Fraser (smfr)
Comment 3 2015-07-17 13:37:24 PDT
Comment on attachment 256982 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=256982&action=review r+ on the code change, but the test needs tweaking. > LayoutTests/compositing/video/poster.html:13 > +<p>This test makes sure that video posters are displayed even after the source is loaded</p> > +<video id="v" src="../resources/video.mp4" poster="../resources/simple_image.png"></video> Doesn't this bug only reproduce if controls are shown, or the video is composited for other reasons? You should use the scripts used by media tests to load the video, too.
Eric Carlson
Comment 4 2015-07-17 13:55:24 PDT
Comment on attachment 256982 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=256982&action=review >> LayoutTests/compositing/video/poster.html:13 >> +<video id="v" src="../resources/video.mp4" poster="../resources/simple_image.png"></video> > > Doesn't this bug only reproduce if controls are shown, or the video is composited for other reasons? > > You should use the scripts used by media tests to load the video, too. Yes, please include media-file.js and use findMediaFile.
Myles C. Maxfield
Comment 5 2015-07-17 14:37:42 PDT
Created attachment 256990 [details] Patch
Myles C. Maxfield
Comment 6 2015-07-17 15:16:15 PDT
Created attachment 256997 [details] Patch
Myles C. Maxfield
Comment 7 2015-07-17 16:02:25 PDT
Committed r186968: <http://trac.webkit.org/changeset/186968>
Note You need to log in before you can comment on or make changes to this bug.