[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

andersca at apple.com andersca at apple.com
Wed Dec 22 14:34:52 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit a647df456184a13b17059ae503669aa28ba7913c
Author: andersca at apple.com <andersca at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Wed Oct 13 18:01:11 2010 +0000

    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):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@69672 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebKit2/ChangeLog b/WebKit2/ChangeLog
index 1365846..db183de 100644
--- a/WebKit2/ChangeLog
+++ b/WebKit2/ChangeLog
@@ -1,3 +1,14 @@
+2010-10-13  Anders Carlsson  <andersca at 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 at apple.com>
 
         Let WebCore handle scrolling when the spacebar is pressed on Windows
diff --git a/WebKit2/UIProcess/Launcher/mac/ProcessLauncherMac.mm b/WebKit2/UIProcess/Launcher/mac/ProcessLauncherMac.mm
index 35620d6..fbeab49 100644
--- a/WebKit2/UIProcess/Launcher/mac/ProcessLauncherMac.mm
+++ b/WebKit2/UIProcess/Launcher/mac/ProcessLauncherMac.mm
@@ -114,7 +114,14 @@ void ProcessLauncher::launchProcess()
     posix_spawnattr_t attr;
     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.
     cpu_type_t architecture = m_launchOptions.architecture;
@@ -126,7 +133,9 @@ void ProcessLauncher::launchProcess()
     posix_spawnattr_setbinpref_np(&attr, 1, cpuTypes, &outCount);
 
     // 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;
     int result = posix_spawn(&processIdentifier, path, 0, &attr, (char *const*)args, *_NSGetEnviron());

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list