Attachment 340341 Details for Bug 185619 – Patch

[patch] Patch

bug-185619-20180514123002.patch (text/plain), 3.81 KB, created by Jeremy Jones on 2018-05-14 12:30:03 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Jones

Created: 2018-05-14 12:30:03 PDT

Size: 3.81 KB

patch

obsolete

>Subversion Revision: 231175
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index f5230246676a60de86427223b4de85689e461c48..db2271346ecb0116ec63e1a38fb8e5985664618f 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,24 @@
>+2018-05-14  Jeremy Jones  <jeremyj@apple.com>
>+
>+        Exit fullscreen when javascript alerts are presented.
>+        https://bugs.webkit.org/show_bug.cgi?id=185619
>+        rdar://problem/35724264
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Prevent users from being trapped in fullscreen by alert cycles.
>+        Prevent fullscreen from becoming unresponsive due to alerts hidden behind fullscreen.
>+
>+        Fullscreen exit is initiated here in the UI process so that fullscreen UI is torn down immediately,
>+        before the alert is shown.
>+
>+        * UIProcess/WebPageProxy.cpp:
>+        (WebKit::WebPageProxy::exitFullscreenImmediately):
>+        (WebKit::WebPageProxy::runJavaScriptAlert):
>+        (WebKit::WebPageProxy::runJavaScriptConfirm):
>+        (WebKit::WebPageProxy::runJavaScriptPrompt):
>+        * UIProcess/WebPageProxy.h:
>+
> 2018-04-30  Alex Christensen  <achristensen@webkit.org>
> 
>         Add WKUIDelegatePrivate equivalent of WKPageContextMenuClient getContextMenuFromProposedMenuAsync
>diff --git a/Source/WebKit/UIProcess/WebPageProxy.cpp b/Source/WebKit/UIProcess/WebPageProxy.cpp
>index c9b48cb8fa9294568aded7be72fc2a3b0d210551..f2c0246d33083b25dae2de4fbb9d0ae97767a778 100644
>--- a/Source/WebKit/UIProcess/WebPageProxy.cpp
>+++ b/Source/WebKit/UIProcess/WebPageProxy.cpp
>@@ -4190,6 +4190,19 @@ void WebPageProxy::showPage()
>     m_uiClient->showPage(this);
> }
> 
>+void WebPageProxy::exitFullscreenImmediately()
>+{
>+#if ENABLE(FULLSCREEN_API)
>+    if (fullScreenManager())
>+        fullScreenManager()->close();
>+#endif
>+
>+#if (PLATFORM(IOS) && HAVE(AVKIT)) || (PLATFORM(MAC) && ENABLE(VIDEO_PRESENTATION_MODE))
>+    if (videoFullscreenManager())
>+        videoFullscreenManager()->requestHideAndExitFullscreen();
>+#endif
>+}
>+
> void WebPageProxy::fullscreenMayReturnToInline()
> {
>     m_uiClient->fullscreenMayReturnToInline(this);
>@@ -4219,6 +4232,8 @@ void WebPageProxy::runJavaScriptAlert(uint64_t frameID, const SecurityOriginData
>     WebFrameProxy* frame = m_process->webFrame(frameID);
>     MESSAGE_CHECK(frame);
> 
>+    exitFullscreenImmediately();
>+
>     // Since runJavaScriptAlert() can spin a nested run loop we need to turn off the responsiveness timer.
>     m_process->responsivenessTimer().stop();
> 
>@@ -4236,6 +4251,8 @@ void WebPageProxy::runJavaScriptConfirm(uint64_t frameID, const SecurityOriginDa
>     WebFrameProxy* frame = m_process->webFrame(frameID);
>     MESSAGE_CHECK(frame);
> 
>+    exitFullscreenImmediately();
>+
>     // Since runJavaScriptConfirm() can spin a nested run loop we need to turn off the responsiveness timer.
>     m_process->responsivenessTimer().stop();
> 
>@@ -4254,6 +4271,8 @@ void WebPageProxy::runJavaScriptPrompt(uint64_t frameID, const SecurityOriginDat
>     WebFrameProxy* frame = m_process->webFrame(frameID);
>     MESSAGE_CHECK(frame);
> 
>+    exitFullscreenImmediately();
>+
>     // Since runJavaScriptPrompt() can spin a nested run loop we need to turn off the responsiveness timer.
>     m_process->responsivenessTimer().stop();
> 
>diff --git a/Source/WebKit/UIProcess/WebPageProxy.h b/Source/WebKit/UIProcess/WebPageProxy.h
>index 9c9442909ebbe069ed043dd71ed464cc56cb45c3..633a7457c520111f911240813097a667182f7f59 100644
>--- a/Source/WebKit/UIProcess/WebPageProxy.h
>+++ b/Source/WebKit/UIProcess/WebPageProxy.h
>@@ -356,6 +356,7 @@ public:
>     bool addsVisitedLinks() const { return m_addsVisitedLinks; }
>     void setAddsVisitedLinks(bool addsVisitedLinks) { m_addsVisitedLinks = addsVisitedLinks; }
> 
>+    void exitFullscreenImmediately();
>     void fullscreenMayReturnToInline();
>     void didEnterFullscreen();
>     void didExitFullscreen();

Actions: View | Formatted Diff | Diff

Attachments on bug 185619: 340341 | 340367 | 341274 | 341280 | 341765