Attachment 341525 Details for Bug 186065 – Patch

[patch] Patch

bug-186065-20180529152020.patch (text/plain), 8.60 KB, created by Jer Noble on 2018-05-29 15:20:21 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Jer Noble

Created: 2018-05-29 15:20:21 PDT

Size: 8.60 KB

patch

obsolete

>Subversion Revision: 232115
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index d5837e30be10e2c7d609698efb31d1b81c147902..d923930a148a5fbe0635c93b065528b70ab8f4de 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,29 @@
>+2018-05-29  Jer Noble  <jer.noble@apple.com>
>+
>+        Auto-pip should use main content heuristic.
>+        https://bugs.webkit.org/show_bug.cgi?id=186065
>+        <rdar://problem/35862502>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Make the m_pipStandbyElement clearable, which will allow the auto-pip mechanism to be torn down. Add
>+        a WebProcess-side notification when the main content changes, to facilitate this.
>+
>+        * WebProcess/FullScreen/WebFullScreenManager.cpp:
>+        (WebKit::WebFullScreenManager::videoControlsManagerDidChange):
>+        (WebKit::WebFullScreenManager::setPIPStandbyElement):
>+        (WebKit::WebFullScreenManager::didEnterFullScreen):
>+        (WebKit::WebFullScreenManager::willExitFullScreen):
>+        * WebProcess/FullScreen/WebFullScreenManager.h:
>+        * WebProcess/WebPage/WebPage.cpp:
>+        (WebKit::WebPage::videoControlsManagerDidChange):
>+        * WebProcess/WebPage/WebPage.h:
>+        * WebProcess/cocoa/PlaybackSessionManager.h:
>+        * WebProcess/cocoa/PlaybackSessionManager.mm:
>+        (WebKit::PlaybackSessionManager::setUpPlaybackControlsManager):
>+        (WebKit::PlaybackSessionManager::clearPlaybackControlsManager):
>+        (WebKit::PlaybackSessionManager::currentPlaybackControlsElement const):
>+
> 2018-05-23  Carlos Garcia Campos  <cgarcia@igalia.com>
> 
>         [GTK] WebDriver: implement AutomationSessionClient::didDisconnectFromRemote
>diff --git a/Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.cpp b/Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.cpp
>index 963f1b9d5f666a1ce51d7a73568f1881c5ab225a..a33d39fba674118c9a4f89f0d8a5b1e53cdf0f08 100644
>--- a/Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.cpp
>+++ b/Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.cpp
>@@ -28,6 +28,7 @@
> #if ENABLE(FULLSCREEN_API)
> 
> #include "Connection.h"
>+#include "PlaybackSessionManager.h"
> #include "WebCoreArgumentCoders.h"
> #include "WebFrame.h"
> #include "WebFullScreenManagerProxyMessages.h"
>@@ -81,6 +82,28 @@ WebCore::Element* WebFullScreenManager::element()
>     return m_element.get(); 
> }
> 
>+void WebFullScreenManager::videoControlsManagerDidChange()
>+{
>+#if ENABLE(VIDEO)
>+    auto* currentPlaybackControlsElement = m_page->playbackSessionManager().currentPlaybackControlsElement();
>+    setPIPStandbyElement(is<HTMLVideoElement>(currentPlaybackControlsElement) ? downcast<HTMLVideoElement>(currentPlaybackControlsElement) : nullptr);
>+#endif
>+}
>+
>+void WebFullScreenManager::setPIPStandbyElement(WebCore::HTMLVideoElement* pipStandbyElement)
>+{
>+    if (pipStandbyElement == m_pipStandbyElement)
>+        return;
>+
>+    if (m_pipStandbyElement)
>+        m_pipStandbyElement->setVideoFullscreenStandby(false);
>+
>+    m_pipStandbyElement = pipStandbyElement;
>+
>+    if (m_pipStandbyElement)
>+        m_pipStandbyElement->setVideoFullscreenStandby(true);
>+}
>+
> void WebFullScreenManager::didReceiveMessage(IPC::Connection& connection, IPC::Decoder& decoder)
> {
>     didReceiveWebFullScreenManagerMessage(connection, decoder);
>@@ -126,9 +149,8 @@ void WebFullScreenManager::didEnterFullScreen()
>     m_element->document().webkitDidEnterFullScreenForElement(m_element.get());
> 
> #if ENABLE(VIDEO)
>-    m_pipStandbyElement = descendantsOfType<HTMLVideoElement>(*m_element).first();
>-    if (m_pipStandbyElement)
>-        m_pipStandbyElement->setVideoFullscreenStandby(true);
>+    auto* currentPlaybackControlsElement = m_page->playbackSessionManager().currentPlaybackControlsElement();
>+    setPIPStandbyElement(is<HTMLVideoElement>(currentPlaybackControlsElement) ? downcast<HTMLVideoElement>(currentPlaybackControlsElement) : nullptr);
> #endif
> }
> 
>@@ -137,9 +159,7 @@ void WebFullScreenManager::willExitFullScreen()
>     ASSERT(m_element);
> 
> #if ENABLE(VIDEO)
>-    if (m_pipStandbyElement)
>-        m_pipStandbyElement->setVideoFullscreenStandby(false);
>-    m_pipStandbyElement = nullptr;
>+    setPIPStandbyElement(nullptr);
> #endif
> 
>     m_finalFrame = screenRectOfContents(m_element.get());
>diff --git a/Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.h b/Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.h
>index cd9478037c4f1e414b63c9ab400a27d207aa99fc..c170fa2aeaa72e5cf1f30261969de4228da05a79 100644
>--- a/Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.h
>+++ b/Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.h
>@@ -65,11 +65,15 @@ public:
> 
>     WebCore::Element* element();
> 
>+    void videoControlsManagerDidChange();
>+
>     void close();
> 
> protected:
>     WebFullScreenManager(WebPage*);
> 
>+    void setPIPStandbyElement(WebCore::HTMLVideoElement*);
>+
>     void setAnimatingFullScreen(bool);
>     void requestExitFullScreen();
>     void saveScrollPosition();
>diff --git a/Source/WebKit/WebProcess/WebPage/WebPage.cpp b/Source/WebKit/WebProcess/WebPage/WebPage.cpp
>index c1a313e807234b3f8614e2bb965678904d8d398d..f163e84b640fa461803124ab0dc6adae1e5e74d1 100644
>--- a/Source/WebKit/WebProcess/WebPage/WebPage.cpp
>+++ b/Source/WebKit/WebProcess/WebPage/WebPage.cpp
>@@ -3296,6 +3296,15 @@ VideoFullscreenManager& WebPage::videoFullscreenManager()
>         m_videoFullscreenManager = VideoFullscreenManager::create(*this, playbackSessionManager());
>     return *m_videoFullscreenManager;
> }
>+
>+void WebPage::videoControlsManagerDidChange()
>+{
>+#if ENABLE(FULLSCREEN_API)
>+    if (auto* manager = fullScreenManager())
>+        manager->videoControlsManagerDidChange();
>+#endif
>+}
>+
> #endif
> 
> #if PLATFORM(IOS)
>diff --git a/Source/WebKit/WebProcess/WebPage/WebPage.h b/Source/WebKit/WebProcess/WebPage/WebPage.h
>index 67effdb8443eacca6bb9ea6260a9575bc3877283..03346c3e0c951dabfb7aba00241aa4769c1384c1 100644
>--- a/Source/WebKit/WebProcess/WebPage/WebPage.h
>+++ b/Source/WebKit/WebProcess/WebPage/WebPage.h
>@@ -298,6 +298,7 @@ public:
> #if PLATFORM(IOS) || (PLATFORM(MAC) && ENABLE(VIDEO_PRESENTATION_MODE))
>     PlaybackSessionManager& playbackSessionManager();
>     VideoFullscreenManager& videoFullscreenManager();
>+    void videoControlsManagerDidChange();
> #endif
> 
> #if PLATFORM(IOS)
>diff --git a/Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.h b/Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.h
>index 7f406854cac6f2a65dac29081744581111f25336..d297dac2297fff9ab88f6180b9e9d928e502812f 100644
>--- a/Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.h
>+++ b/Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.h
>@@ -109,6 +109,8 @@ public:
>     void clearPlaybackControlsManager();
>     uint64_t contextIdForMediaElement(WebCore::HTMLMediaElement&);
> 
>+    WebCore::HTMLMediaElement* currentPlaybackControlsElement() const;
>+
> protected:
>     friend class PlaybackSessionInterfaceContext;
>     friend class VideoFullscreenManager;
>diff --git a/Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.mm b/Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.mm
>index c5df31e0fce940b5743b1fdf8f7f3131ad83fd3a..b010d3076b0b30c98050319091bf7093f8c027b4 100644
>--- a/Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.mm
>+++ b/Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.mm
>@@ -278,19 +278,21 @@ void PlaybackSessionManager::setUpPlaybackControlsManager(WebCore::HTMLMediaElem
>     }
> 
>     addClientForContext(m_controlsManagerContextId);
>+
>+    m_page->videoControlsManagerDidChange();
>     m_page->send(Messages::PlaybackSessionManagerProxy::SetUpPlaybackControlsManagerWithID(m_controlsManagerContextId), m_page->pageID());
> }
> 
> void PlaybackSessionManager::clearPlaybackControlsManager()
> {
>-#if PLATFORM(MAC)
>     if (!m_controlsManagerContextId)
>         return;
> 
>     removeClientForContext(m_controlsManagerContextId);
>     m_controlsManagerContextId = 0;
>+
>+    m_page->videoControlsManagerDidChange();
>     m_page->send(Messages::PlaybackSessionManagerProxy::ClearPlaybackControlsManager(), m_page->pageID());
>-#endif
> }
> 
> uint64_t PlaybackSessionManager::contextIdForMediaElement(WebCore::HTMLMediaElement& mediaElement)
>@@ -301,6 +303,18 @@ uint64_t PlaybackSessionManager::contextIdForMediaElement(WebCore::HTMLMediaElem
>     return contextId;
> }
> 
>+WebCore::HTMLMediaElement* PlaybackSessionManager::currentPlaybackControlsElement() const
>+{
>+    if (!m_controlsManagerContextId)
>+        return nullptr;
>+
>+    auto iter = m_contextMap.find(m_controlsManagerContextId);
>+    if (iter == m_contextMap.end())
>+        return nullptr;
>+
>+    return std::get<0>(iter->value)->mediaElement();
>+}
>+
> #pragma mark Interface to PlaybackSessionInterfaceContext:
> 
> void PlaybackSessionManager::resetMediaState(uint64_t contextId)

Actions: View | Formatted Diff | Diff

Attachments on bug 186065: 341525 | 341535