Changeset 69672 in webkit

Timestamp:

Oct 13, 2010 11:01:11 AM (14 years ago)

Author:

andersca@apple.com

Message:

Allow all signals to be delivered to the web process
​https://bugs.webkit.org/show_bug.cgi?id=47602
<rdar://problem/8546399>

Reviewed by Sam Weinig.

• UIProcess/Launcher/mac/ProcessLauncherMac.mm:

(WebKit::ProcessLauncher::launchProcess):

Location:

trunk/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/Launcher/mac/ProcessLauncherMac.mm (modified) (2 diffs)

trunk/WebKit2/ChangeLog

@@ +1 @@
+2010-10-13  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by Sam Weinig.
+
+        Allow all signals to be delivered to the web process
+        https://bugs.webkit.org/show_bug.cgi?id=47602
+        <rdar://problem/8546399>
+
+        * UIProcess/Launcher/mac/ProcessLauncherMac.mm:
+        (WebKit::ProcessLauncher::launchProcess):
+
 2010-10-12  Adam Roben  <aroben@apple.com>
 

trunk/WebKit2/UIProcess/Launcher/mac/ProcessLauncherMac.mm

@@ -115 +115 @@
     posix_spawnattr_init(&attr);
 
-    // FIXME: Should we restore signals here?
+    short flags = 0;
+
+    // We want our process to receive all signals.
+    sigset_t signalMaskSet;
+    sigemptyset(&signalMaskSet);
+
+    posix_spawnattr_setsigmask(&attr, &signalMaskSet);
+    flags |= POSIX_SPAWN_SETSIGMASK;
 
     // Determine the architecture to use.
@@ -127 +134 @@
 
     // Start suspended so we can set up the termination notification handler.
-    posix_spawnattr_setflags(&attr, POSIX_SPAWN_START_SUSPENDED);
+    flags |= POSIX_SPAWN_START_SUSPENDED;
+
+    posix_spawnattr_setflags(&attr, flags);
 
     pid_t processIdentifier;