[mac] LayoutTest media/modern-media-controls/airplay-button/airplay-button.html is...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Oct 2016 14:06:29 +0000 (14:06 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Oct 2016 14:06:29 +0000 (14:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=163248

Patch by Antoine Quint <graouts@apple.com> on 2016-10-11
Reviewed by Antoine Quint.

Wait until all icon images are loaded before running the assertions. We can determine whether
a button's icon image is loaded by checking it's been sized.

* TestExpectations:
* media/modern-media-controls/airplay-button/airplay-button.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@207135 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/TestExpectations
LayoutTests/media/modern-media-controls/airplay-button/airplay-button.html

index 258ab79..a60c7e3 100644 (file)
@@ -1,5 +1,18 @@
 2016-10-11  Antoine Quint  <graouts@apple.com>
 
+        [mac] LayoutTest media/modern-media-controls/airplay-button/airplay-button.html is a flaky failure
+        https://bugs.webkit.org/show_bug.cgi?id=163248
+
+        Reviewed by Antoine Quint.
+
+        Wait until all icon images are loaded before running the assertions. We can determine whether
+        a button's icon image is loaded by checking it's been sized.
+
+        * TestExpectations:
+        * media/modern-media-controls/airplay-button/airplay-button.html:
+
+2016-10-11  Antoine Quint  <graouts@apple.com>
+
         [Modern Media Controls] Buttons container
         https://bugs.webkit.org/show_bug.cgi?id=163238
         <rdar://problem/28701864>
index 0888272..7bd471c 100644 (file)
@@ -986,5 +986,4 @@ webkit.org/b/162414 imported/w3c/web-platform-tests/resource-timing/idlharness.h
 # Only iOS has implemented lettepress.
 fast/text/letterpress-different.html [ ImageOnlyFailure ]
 
-webkit.org/b/163248 media/modern-media-controls/airplay-button/airplay-button.html [ Failure ]
 webkit.org/b/163249 media/modern-media-controls/play-pause-button/play-pause-button.html [ Failure ]
index 00cf8a3..766a659 100644 (file)
@@ -31,6 +31,9 @@ const iOSAirplayButton = new AirplayButton({ layoutTraits: LayoutTraits.iOS });
 
 scheduler.frameDidFire = function()
 {
+    if ([airplayButton, macOSFullscreenAirplayButton, iOSAirplayButton].some(button => button.width == 0))
+        return;
+
     shouldBeTrue('airplayButton.element.style.webkitMaskImage.includes("macOS/airplay@")');
     shouldBeTrue('macOSFullscreenAirplayButton.element.style.webkitMaskImage.includes("macOS/airplay-fullscreen@")');
     shouldBeTrue('iOSAirplayButton.element.style.webkitMaskImage.includes("iOS/airplay@")');