REGRESSION: LayoutTest media/modern-media-controls/placard-support/placard-support...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Jun 2017 22:47:00 +0000 (22:47 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Jun 2017 22:47:00 +0000 (22:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=173725

Patch by Antoine Quint <graouts@apple.com> on 2017-06-22
Reviewed by Dean Jackson.

On WK1, this test enters fullscreen rather than PiP (which is unexpected, see webkit.org/b/173729) and thus the button
to exit PiP could not be clicked. But we don't need a button click to exit PiP, so instead we just call a method to
toggle it.

* media/modern-media-controls/placard-support/placard-support-pip.html:

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

LayoutTests/ChangeLog
LayoutTests/media/modern-media-controls/placard-support/placard-support-pip.html

index fb67164..07bee4b 100644 (file)
@@ -1,3 +1,16 @@
+2017-06-22  Antoine Quint  <graouts@apple.com>
+
+        REGRESSION: LayoutTest media/modern-media-controls/placard-support/placard-support-pip.html is a flaky timeout on Sierra
+        https://bugs.webkit.org/show_bug.cgi?id=173725
+
+        Reviewed by Dean Jackson.
+
+        On WK1, this test enters fullscreen rather than PiP (which is unexpected, see webkit.org/b/173729) and thus the button
+        to exit PiP could not be clicked. But we don't need a button click to exit PiP, so instead we just call a method to
+        toggle it.
+
+        * media/modern-media-controls/placard-support/placard-support-pip.html:
+
 2017-06-22  Ryan Haddad  <ryanhaddad@apple.com>
 
         Mark perf/object-keys.html as flaky on iOS.
index 2742ae7..1aafa08 100644 (file)
@@ -35,15 +35,13 @@ const mediaController = createControls(container, media, null);
 const button = document.body.appendChild(document.createElement("div"));
 button.innerText = "Enter picture-in-picture";
 
-button.addEventListener("click", () => {
-    media.webkitSetPresentationMode(media.webkitPresentationMode == "inline" ? "picture-in-picture" : "inline");
-});
+button.addEventListener("click", togglePiP);
 
 media.addEventListener("webkitpresentationmodechanged", () => {
     if (media.webkitPresentationMode == "picture-in-picture") {
         shouldBe("mediaController.controls.placard", "mediaController.controls.pipPlacard");
         shouldBeTrue("mediaController.controls.children.includes(mediaController.controls.pipPlacard)");
-        clickButton();
+        togglePiP();
     } else {
         shouldBeNull("mediaController.controls.placard");
         shouldBeFalse("mediaController.controls.children.includes(mediaController.controls.pipPlacard)");
@@ -60,6 +58,11 @@ function clickButton() {
     pressOnElement(button);
 }
 
+function togglePiP()
+{
+    media.webkitSetPresentationMode(media.webkitPresentationMode == "inline" ? "picture-in-picture" : "inline");
+}
+
 </script>
 <script src="../../../resources/js-test-post.js"></script>
 </body>