Attachment 360155 Details for Bug 193844 – Patch

[patch] Patch

bug-193844-20190125134207.patch (text/plain), 3.53 KB, created by Alex Christensen on 2019-01-25 13:42:07 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Christensen

Created: 2019-01-25 13:42:07 PST

Size: 3.53 KB

patch

obsolete

>Index: Source/WebKit/ChangeLog
>===================================================================
>--- Source/WebKit/ChangeLog	(revision 240500)
>+++ Source/WebKit/ChangeLog	(working copy)
>@@ -1,3 +1,19 @@
>+2019-01-25  Alex Christensen  <achristensen@webkit.org>
>+
>+        Move logic from ChildProcess::setApplicationIsDaemon to NetworkProcess::initializeProcess
>+        https://bugs.webkit.org/show_bug.cgi?id=193844
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        This simplifies ChildProcess and removes NetworkProcess-specific code from this generic abstraction.
>+
>+        * NetworkProcess/mac/NetworkProcessMac.mm:
>+        (WebKit::NetworkProcess::initializeProcess):
>+        * Shared/ios/ChildProcessIOS.mm:
>+        (WebKit::ChildProcess::setApplicationIsDaemon): Deleted.
>+        * Shared/mac/ChildProcessMac.mm:
>+        (WebKit::ChildProcess::setApplicationIsDaemon): Deleted.
>+
> 2019-01-25  Per Arne Vollan  <pvollan@apple.com>
> 
>         [iOS] Deny mach lookups to services not used.
>Index: Source/WebKit/NetworkProcess/mac/NetworkProcessMac.mm
>===================================================================
>--- Source/WebKit/NetworkProcess/mac/NetworkProcessMac.mm	(revision 240489)
>+++ Source/WebKit/NetworkProcess/mac/NetworkProcessMac.mm	(working copy)
>@@ -47,13 +47,20 @@
> #import <wtf/MemoryPressureHandler.h>
> #import <wtf/text/WTFString.h>
> 
>+extern "C" OSStatus SetApplicationIsDaemon(Boolean isDaemon);
>+
> namespace WebKit {
> using namespace WebCore;
> 
> void NetworkProcess::initializeProcess(const ChildProcessInitializationParameters&)
> {
>+#if PLATFORM(MAC) && !PLATFORM(IOSMAC)
>     // Having a window server connection in this process would result in spin logs (<rdar://problem/13239119>).
>-    setApplicationIsDaemon();
>+    OSStatus error = SetApplicationIsDaemon(true);
>+    ASSERT_UNUSED(error, error == noErr);
>+#endif
>+
>+    launchServicesCheckIn();
> }
> 
> void NetworkProcess::initializeProcessName(const ChildProcessInitializationParameters& parameters)
>Index: Source/WebKit/Shared/ios/ChildProcessIOS.mm
>===================================================================
>--- Source/WebKit/Shared/ios/ChildProcessIOS.mm	(revision 240489)
>+++ Source/WebKit/Shared/ios/ChildProcessIOS.mm	(working copy)
>@@ -46,10 +46,6 @@
> 
> namespace WebKit {
> 
>-void ChildProcess::setApplicationIsDaemon()
>-{
>-}
>-
> void ChildProcess::platformInitialize()
> {
>     FloatingPointEnvironment& floatingPointEnvironment = FloatingPointEnvironment::singleton(); 
>Index: Source/WebKit/Shared/mac/ChildProcessMac.mm
>===================================================================
>--- Source/WebKit/Shared/mac/ChildProcessMac.mm	(revision 240489)
>+++ Source/WebKit/Shared/mac/ChildProcessMac.mm	(working copy)
>@@ -76,9 +76,6 @@ typedef bool (^LSServerConnectionAllowed
> extern "C" void _LSSetApplicationLaunchServicesServerConnectionStatus(uint64_t flags, LSServerConnectionAllowedBlock block);
> extern "C" CFDictionaryRef _LSApplicationCheckIn(LSSessionID sessionID, CFDictionaryRef applicationInfo);
> 
>-extern "C" OSStatus SetApplicationIsDaemon(Boolean isDaemon);
>-
>-
> namespace WebKit {
> using namespace WebCore;
> 
>@@ -152,17 +149,6 @@ static void initializeTimerCoalescingPol
>     ASSERT_UNUSED(kr, kr == KERN_SUCCESS);
> }
> 
>-void ChildProcess::setApplicationIsDaemon()
>-{
>-#if !PLATFORM(IOSMAC)
>-    OSStatus error = SetApplicationIsDaemon(true);
>-    ASSERT_UNUSED(error, error == noErr);
>-#endif
>-
>-    // FIXME: Is this needed in iOSMac?
>-    launchServicesCheckIn();
>-}
>-
> void ChildProcess::launchServicesCheckIn()
> {
>     _LSSetApplicationLaunchServicesServerConnectionStatus(0, 0);

Actions: View | Formatted Diff | Diff

Attachments on bug 193844: 360155 | 360178