Changeset 211302 in webkit

Timestamp:

Jan 27, 2017, 2:23:32 PM (10 years ago)

Author:

jer.noble@apple.com

Message:

media/track/track-in-band-style.html is flaky
​https://bugs.webkit.org/show_bug.cgi?id=167516

Reviewed by Eric Carlson.

Update the test to use 'cuechange' rather than a 200ms timer.

• media/track/track-in-band-style-expected.txt:
• media/track/track-in-band-style.html:
• platform/mac/TestExpectations:

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• media/track/track-in-band-style-expected.txt (modified) (1 diff)
• media/track/track-in-band-style.html (modified) (1 diff)
• platform/mac/TestExpectations (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2017-01-27  Jer Noble  <jer.noble@apple.com>
+
+        media/track/track-in-band-style.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=167516
+
+        Reviewed by Eric Carlson.
+
+        Update the test to use 'cuechange' rather than a 200ms timer.
+
+        * media/track/track-in-band-style-expected.txt:
+        * media/track/track-in-band-style.html:
+        * platform/mac/TestExpectations:
+
 2017-01-27  Youenn Fablet  <youenn@apple.com>
 

trunk/LayoutTests/media/track/track-in-band-style-expected.txt

@@ -11 +11 @@
 EVENT(change)
 RUN(video.play())
-EVENT(seeked)
+EVENT(cuechange)
 
 ** Test current cue colors

trunk/LayoutTests/media/track/track-in-band-style.html

@@ -35 +35 @@
                 {
                     run("video.play()");
-                    setTimeout(function() { video.pause(); video.currentTime = 0.3; }, 1000);
-                    waitForEvent('seeked', function() { setTimeout(testCues, 200) });
+                    setTimeout(function() {
+                        video.pause();
+                        video.currentTime = 0.3;
+                        waitForEventOn(inbandTrack1, 'cuechange', testCues);
+                    }, 1000);
                 }
     

trunk/LayoutTests/platform/mac/TestExpectations

@@ -1342 +1342 @@
 [ Sierra+ ] fast/canvas/canvas-imageSmoothingEnabled-repaint.html [ Pass Failure ]
 
-# <rdar://problem/23601055>
-[ Sierra+ ] media/track/track-in-band-style.html [ Pass Failure ]
-
 # Sierra supports true font feature coverage queries. We use the -coverage versions of these tests instead.
 [ Sierra+ ] css3/font-variant-small-caps-synthesis.html [ ImageOnlyFailure ]