Title: [149197] trunk/LayoutTests
Revision
149197
Author
eric.carl...@apple.com
Date
2013-04-26 11:10:38 -0700 (Fri, 26 Apr 2013)

Log Message

[Mac] media/track/track-mode.html sometimes trigger InvalidStateError
https://bugs.webkit.org/show_bug.cgi?id=115045

Reviewed by Alexey Proskuryakov.

* media/track/track-mode.html: Don't assume that the video element is ready for seeking
    after a 100ms timeout.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (149196 => 149197)


--- trunk/LayoutTests/ChangeLog	2013-04-26 17:51:13 UTC (rev 149196)
+++ trunk/LayoutTests/ChangeLog	2013-04-26 18:10:38 UTC (rev 149197)
@@ -1,3 +1,13 @@
+2013-04-26  Eric Carlson  <eric.carl...@apple.com>
+
+        [Mac] media/track/track-mode.html sometimes trigger InvalidStateError
+        https://bugs.webkit.org/show_bug.cgi?id=115045
+
+        Reviewed by Alexey Proskuryakov.
+
+        * media/track/track-mode.html: Don't assume that the video element is ready for seeking
+            after a 100ms timeout.
+
 2013-04-26  David Kilzer  <ddkil...@apple.com>
 
         Sort jsc-test-list

Modified: trunk/LayoutTests/media/track/track-mode.html (149196 => 149197)


--- trunk/LayoutTests/media/track/track-mode.html	2013-04-26 17:51:13 UTC (rev 149196)
+++ trunk/LayoutTests/media/track/track-mode.html	2013-04-26 18:10:38 UTC (rev 149197)
@@ -45,14 +45,23 @@
                 consoleWrite("<b>++ 'disabled'</b>");
                 setMode("disabled");
 
+                waitForEvent('cuechange', cuechange);
+                video.addEventListener('playing', playing);
                 video.src = "" "../content/test");
                 video.play();
-                // Wait for end of first cue (no events should fire while track is disabled).
-                setTimeout(testHiddenAndShowing, 100);
+
                 consoleWrite("++ 0 events expected while mode = disabled");
                 consoleWrite("");
             }
             
+            function playing()
+            {
+                video.removeEventListener('playing', playing);
+
+                // Play past the end of the first cue, no events should fire because track is disabled.
+                setTimeout(testHiddenAndShowing, 100);
+            }
+
             function testHiddenAndShowing()
             {
                 video.pause();
@@ -64,20 +73,19 @@
                 consoleWrite("<b>++ 'showing'</b>");
                 setMode("showing");
 
-                consoleWrite("++ at least " + textTrack.cues.length + " events expected while mode = showing");
+                consoleWrite("<b>++ at least " + textTrack.cues.length + " events expected while mode = showing</b>");
                 video.play();
             }
             
-            waitForEvent('cuechange',
-                function (event)
-                {
-                    cueCount++;
-                    if (cueCount == textTrack.cues.length)
-                        endTest();
+            function cuechange(event)
+            {
+                cueCount++;
+                if (cueCount == textTrack.cues.length) {
+                    endTest();
+                    video.pause();
+                }
+            }
 
-                }
-            );
-            
         </script>
     </head>
     <body>
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to