media/video-fullscreen-restriction-removed.html is flaky on Mac

https://bugs.webkit.org/show_bug.cgi?id=156983 Reviewed by Darin Adler. * media/video-fullscreen-restriction-removed-expected.txt: * media/video-fullscreen-restriction-removed.html: Start the test on 'canplaythrough' instead of 'loadedmetadata' so playback can begin immediately. Canonical link: https://commits.webkit.org/175190@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@200137 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebKit · Apr 27, 2016 · 3747e4786fa68ae3424ed960593c6d80442be7bb · 3747e47
1 parent 5e79fb3
commit 3747e4786fa68ae3424ed960593c6d80442be7bb
Showing with 14 additions and 5 deletions.

+11 −0 LayoutTests/ChangeLog

+1 −1 LayoutTests/media/video-fullscreen-restriction-removed-expected.txt

+2 −2 LayoutTests/media/video-fullscreen-restriction-removed.html

+0 −2 LayoutTests/platform/mac/TestExpectations
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
@@ -1,3 +1,14 @@
+2016-04-27  Eric Carlson  <eric.carlson@apple.com>
+
+        media/video-fullscreen-restriction-removed.html is flaky on Mac
+        https://bugs.webkit.org/show_bug.cgi?id=156983
+
+        Reviewed by Darin Adler.
+
+        * media/video-fullscreen-restriction-removed-expected.txt:
+        * media/video-fullscreen-restriction-removed.html: Start the test on 'canplaythrough' instead
+          of 'loadedmetadata' so playback can begin immediately.
+
 2016-04-27  Ryan Haddad  <ryanhaddad@apple.com>
 
         Add ios-simulator baseline for LayoutTest fast/text/hyphenate-avoid-orphaned-word.html

diff --git a/LayoutTests/media/video-fullscreen-restriction-removed-expected.txt b/LayoutTests/media/video-fullscreen-restriction-removed-expected.txt
@@ -5,7 +5,7 @@ RUN(internals.settings.setAllowsInlineMediaPlayback(false))
 RUN(internals.setMediaElementRestrictions(video, "RequireUserGestureForFullscreen"))
 * set video.src
 
-EVENT(loadedmetadata)
+EVENT(canplaythrough)
 
 * video.play() should fail 
 RUN(video.play())

diff --git a/LayoutTests/media/video-fullscreen-restriction-removed.html b/LayoutTests/media/video-fullscreen-restriction-removed.html
@@ -16,7 +16,7 @@
                 run('internals.settings.setAllowsInlineMediaPlayback(false)');
                 run('internals.setMediaElementRestrictions(video, "RequireUserGestureForFullscreen")');
 
-                waitForEvent('loadedmetadata', loadedmetadata)
+                waitForEvent('canplaythrough', canplaythrough)
                 waitForEvent('webkitfullscreenchange', fullscreenChanged, false, true, document)
                 waitForEvent('click', clickHandler, false, true, document.getElementById('button'))
 
@@ -25,7 +25,7 @@
                 consoleWrite('');
             }
 
-            function loadedmetadata()
+            function canplaythrough()
             {
                 consoleWrite('');
                 if (window.eventSender) {

diff --git a/LayoutTests/platform/mac/TestExpectations b/LayoutTests/platform/mac/TestExpectations
@@ -1290,8 +1290,6 @@ webkit.org/b/152009 [ Debug ElCapitan ] fast/canvas/canvas-too-large-to-draw.htm
 
 webkit.org/b/153086 sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.3/15.1.3.3_encodeURI/S15.1.3.3_A2.4_T2.html [ Pass Crash ]
 
-[ Yosemite ] media/video-fullscreen-restriction-removed.html [ Skip ]
-
 webkit.org/b/153752 [ Yosemite ] http/tests/plugins/visible_plugins.html [ Skip ]
 
 webkit.org/b/153809 transitions/clip-path-transitions.html [ Pass Failure ]