Title: [211689] branches/safari-603-branch/LayoutTests

Diff

Modified: branches/safari-603-branch/LayoutTests/ChangeLog (211688 => 211689)


--- branches/safari-603-branch/LayoutTests/ChangeLog	2017-02-05 23:04:48 UTC (rev 211688)
+++ branches/safari-603-branch/LayoutTests/ChangeLog	2017-02-06 04:03:20 UTC (rev 211689)
@@ -1,3 +1,20 @@
+2017-02-02  Matthew Hanson  <matthew_han...@apple.com>
+
+        Merge r211302. rdar://problem/23601055
+
+    2017-01-27  Jer Noble  <jer.no...@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-31  Matthew Hanson  <matthew_han...@apple.com>
 
         Merge r211387. rdar://problem/29500273

Modified: branches/safari-603-branch/LayoutTests/media/track/track-in-band-style-expected.txt (211688 => 211689)


--- branches/safari-603-branch/LayoutTests/media/track/track-in-band-style-expected.txt	2017-02-05 23:04:48 UTC (rev 211688)
+++ branches/safari-603-branch/LayoutTests/media/track/track-in-band-style-expected.txt	2017-02-06 04:03:20 UTC (rev 211689)
@@ -10,7 +10,7 @@
 RUN(inbandTrack1.mode = 'showing')
 EVENT(change)
 RUN(video.play())
-EVENT(seeked)
+EVENT(cuechange)
 
 ** Test current cue colors
 EXPECTED (getComputedStyle(cueDisplayElement).color == 'rgb(255, 255, 255)') OK

Modified: branches/safari-603-branch/LayoutTests/media/track/track-in-band-style.html (211688 => 211689)


--- branches/safari-603-branch/LayoutTests/media/track/track-in-band-style.html	2017-02-05 23:04:48 UTC (rev 211688)
+++ branches/safari-603-branch/LayoutTests/media/track/track-in-band-style.html	2017-02-06 04:03:20 UTC (rev 211689)
@@ -34,8 +34,11 @@
                 function tracksChanged()
                 {
                     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);
                 }
     
                 function canplaythrough()

Modified: branches/safari-603-branch/LayoutTests/platform/mac/TestExpectations (211688 => 211689)


--- branches/safari-603-branch/LayoutTests/platform/mac/TestExpectations	2017-02-05 23:04:48 UTC (rev 211688)
+++ branches/safari-603-branch/LayoutTests/platform/mac/TestExpectations	2017-02-06 04:03:20 UTC (rev 211689)
@@ -1338,9 +1338,6 @@
 # <rdar://problem/23505582> Fails on iMac10,1 and iMac11,1
 [ 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 ]
 [ Sierra+ ] css3/font-variant-petite-caps-synthesis.html [ ImageOnlyFailure ]
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to