REGRESSION(r233926): media/modern-media-controls/media-controller/media-controller...
authorjer.noble@apple.com <jer.noble@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Jul 2018 20:48:43 +0000 (20:48 +0000)
committerjer.noble@apple.com <jer.noble@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Jul 2018 20:48:43 +0000 (20:48 +0000)
https://bugs.webkit.org/show_bug.cgi?id=187813

Reviewed by Jon Lee.

In r233926, we changed the behavior of entering PiP to exit fullscreen only after entering PiP completes. The
test in question will immediately request "inline" presentation mode once the PiP animation begins, and thus
it's asking to "exit fullscreen" when both in standard fullscreen and also in PiP. The fix is not to bail out
early if we're in standard (element) fullscreen, but to allow the remaining steps to complete and exit PiP as
well.

* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::exitFullscreen):

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

Source/WebCore/ChangeLog
Source/WebCore/html/HTMLMediaElement.cpp

index b11ff57..26ddae7 100644 (file)
@@ -1,3 +1,19 @@
+2018-07-19  Jer Noble  <jer.noble@apple.com>
+
+        REGRESSION(r233926): media/modern-media-controls/media-controller/media-controller-inline-to-fullscreen-to-pip-to-inline.html is a TIMEOUT failure
+        https://bugs.webkit.org/show_bug.cgi?id=187813
+
+        Reviewed by Jon Lee.
+
+        In r233926, we changed the behavior of entering PiP to exit fullscreen only after entering PiP completes. The
+        test in question will immediately request "inline" presentation mode once the PiP animation begins, and thus
+        it's asking to "exit fullscreen" when both in standard fullscreen and also in PiP. The fix is not to bail out
+        early if we're in standard (element) fullscreen, but to allow the remaining steps to complete and exit PiP as
+        well.
+
+        * html/HTMLMediaElement.cpp:
+        (WebCore::HTMLMediaElement::exitFullscreen):
+
 2018-07-19  Zalan Bujtas  <zalan@apple.com>
 
         [LFC] Introduce simple line breaker.
index 409052e..44bca6b 100644 (file)
@@ -5958,7 +5958,9 @@ void HTMLMediaElement::exitFullscreen()
     if (document().settings().fullScreenEnabled() && document().webkitCurrentFullScreenElement() == this) {
         if (document().webkitIsFullScreen())
             document().webkitCancelFullScreen();
-        return;
+
+        if (m_videoFullscreenMode == VideoFullscreenModeStandard)
+            return;
     }
 #endif