Attachment 341606 Details for Bug 186093 – Patch

[patch] Patch

bug-186093-20180530150615.patch (text/plain), 3.18 KB, created by Jeremy Jones on 2018-05-30 15:06:16 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Jones

Created: 2018-05-30 15:06:16 PDT

Size: 3.18 KB

patch

obsolete

>Subversion Revision: 231991
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index ef77cb83859a3bc9b332867e0a88dea75276f71a..342217b278081fc26b5a923cdc6772ccf2bd3db3 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,22 @@
>+2018-05-30  Jeremy Jones  <jeremyj@apple.com>
>+
>+        Enable video controls when in element fullscreen.
>+        https://bugs.webkit.org/show_bug.cgi?id=186093
>+        rdar://problem/40144751
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Some pages use webkitRequestFullscreen directly on the video element, without any custom controls.
>+        This will prevent native fullscreeen presentation mode conrols from being used.
>+
>+        This change makes webkitRequestFullscreen on a video element call webKitSetPresentationMode
>+        and log to the console to inform the developer that they should adopt webKitSetPresentationMode.
>+
>+        * dom/Element.h:
>+        * html/HTMLVideoElement.cpp:
>+        (WebCore::HTMLVideoElement::webkitRequestFullscreen):
>+        * html/HTMLVideoElement.h:
>+
> 2018-05-18  Jer Noble  <jer.noble@apple.com>
> 
>         Complete fix for enabling modern EME by default
>diff --git a/Source/WebCore/dom/Element.h b/Source/WebCore/dom/Element.h
>index 1c4db86823c6e41ec5fc3ab3f20194516eaffa06..d7f4d5287a19da6be58708780adf05886eac5148 100644
>--- a/Source/WebCore/dom/Element.h
>+++ b/Source/WebCore/dom/Element.h
>@@ -475,7 +475,7 @@ public:
>     WEBCORE_EXPORT bool containsFullScreenElement() const;
>     void setContainsFullScreenElement(bool);
>     void setContainsFullScreenElementOnAncestorsCrossingFrameBoundaries(bool);
>-    WEBCORE_EXPORT void webkitRequestFullscreen();
>+    WEBCORE_EXPORT virtual void webkitRequestFullscreen();
> #endif
> 
> #if ENABLE(POINTER_LOCK)
>diff --git a/Source/WebCore/html/HTMLVideoElement.cpp b/Source/WebCore/html/HTMLVideoElement.cpp
>index 908ed4ca6836ce320c1c3f9083ecec9b3210c8c9..2b53a40bb40a37e05a9c58af33c27c8d854c0afc 100644
>--- a/Source/WebCore/html/HTMLVideoElement.cpp
>+++ b/Source/WebCore/html/HTMLVideoElement.cpp
>@@ -191,6 +191,15 @@ bool HTMLVideoElement::supportsFullscreen(HTMLMediaElementEnums::VideoFullscreen
> #endif // PLATFORM(IOS)
> }
> 
>+
>+#if ENABLE(FULLSCREEN_API) && PLATFORM(IOS)
>+void HTMLVideoElement::webkitRequestFullscreen()
>+{
>+    document().addConsoleMessage(MessageSource::Media, MessageLevel::Debug, ASCIILiteral("For video element use webkitSetPresentationMode('Fullscreen') instead of webkitRequestFullscreen()."));
>+    webkitSetPresentationMode(HTMLVideoElement::VideoPresentationMode::Fullscreen);
>+}
>+#endif
>+
> unsigned HTMLVideoElement::videoWidth() const
> {
>     if (!player())
>diff --git a/Source/WebCore/html/HTMLVideoElement.h b/Source/WebCore/html/HTMLVideoElement.h
>index cb72b7c84451b73f450d8a38770713ca936fab8d..40a7af351f89511ff353a5d64138fddff933cf66 100644
>--- a/Source/WebCore/html/HTMLVideoElement.h
>+++ b/Source/WebCore/html/HTMLVideoElement.h
>@@ -61,6 +61,10 @@ public:
>     unsigned webkitDroppedFrameCount() const;
> #endif
> 
>+#if ENABLE(FULLSCREEN_API) && PLATFORM(IOS)
>+    void webkitRequestFullscreen() override;
>+#endif
>+
>     // Used by canvas to gain raw pixel access
>     void paintCurrentFrameInContext(GraphicsContext&, const FloatRect&);
>

Actions: View | Formatted Diff | Diff

Attachments on bug 186093: 341570 | 341606 | 343820 | 343823 | 343843 | 344473 | 344497