Attachment 339592 Details for Bug 185082 – Patch

[patch] Patch

bug-185082-20180504140232.patch (text/plain), 1.66 KB, created by Per Arne Vollan on 2018-05-04 14:02:34 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Per Arne Vollan

Created: 2018-05-04 14:02:34 PDT

Size: 1.66 KB

patch

obsolete

>Index: Source/WebKit/ChangeLog
>===================================================================
>--- Source/WebKit/ChangeLog	(revision 231378)
>+++ Source/WebKit/ChangeLog	(working copy)
>@@ -1,3 +1,17 @@
>+2018-05-04  Per Arne Vollan  <pvollan@apple.com>
>+
>+        Shutdown WindowServer connections after checking in with launch services
>+        https://bugs.webkit.org/show_bug.cgi?id=185082
>+        <rdar://problem/39613173>
>+
>+        Reviewed by Brent Fulgham.
>+
>+        When WindowServer access is blocked in the WebContent process, we should shutdown all connections
>+        after checking in with launch services.
>+
>+        * WebProcess/cocoa/WebProcessCocoa.mm:
>+        (WebKit::WebProcess::platformInitializeProcess):
>+
> 2018-05-04  Chris Dumez  <cdumez@apple.com>
> 
>         [iOS] Apps that are not visible may not get suspended if they trigger page loads while in the background
>Index: Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm
>===================================================================
>--- Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm	(revision 231372)
>+++ Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm	(working copy)
>@@ -303,6 +303,8 @@ void WebProcess::platformInitializeProce
>     // Deny the WebContent process access to the WindowServer.
>     // This call will not succeed if there are open WindowServer connections at this point.
>     setApplicationIsDaemon();
>+    // Make sure that we close any WindowServer connections after checking in with Launch Services.
>+    CGSShutdownServerConnections();
> #else
> #if __MAC_OS_X_VERSION_MIN_REQUIRED >= 101400
>     // This call is needed when the WebProcess is not running the NSApplication event loop.

Actions: View | Formatted Diff | Diff

Attachments on bug 185082: 339001 | 339002 | 339592 | 339605