Changeset 81507 in webkit

Timestamp:

Mar 18, 2011 1:57:11 PM (13 years ago)

Author:

kbalazs@webkit.org

Message:

2011-03-18 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Benjamin Poulain.

[Qt][WK2] Need a way to debug the web process after the UI process dies
​https://bugs.webkit.org/show_bug.cgi?id=56116

Allow the web process to outlive it's parent process
in debug builds if the QT_WEBKIT_KEEP_ALIVE_WEB_PROCESS
environment variable is set.

• UIProcess/Launcher/qt/ProcessLauncherQt.cpp: (WebKit::QtWebProcess::setupChildProcess): Used getenv since the prctl call is already in a Linux-only block and the implementation in this file is a candidate for sharing across ports in the future.

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/Launcher/qt/ProcessLauncherQt.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2011-03-18  Balazs Kelemen  <kbalazs@webkit.org>
+
+        Reviewed by Benjamin Poulain.
+
+        [Qt][WK2] Need a way to debug the web process after the UI process dies
+        https://bugs.webkit.org/show_bug.cgi?id=56116
+
+        Allow the web process to outlive it's parent process
+        in debug builds if the QT_WEBKIT_KEEP_ALIVE_WEB_PROCESS
+        environment variable is set.
+
+        * UIProcess/Launcher/qt/ProcessLauncherQt.cpp:
+        (WebKit::QtWebProcess::setupChildProcess): Used getenv
+        since the prctl call is already in a Linux-only block
+        and the implementation in this file is a candidate
+        for sharing across ports in the future.
+
 2011-03-18  Darin Adler  <darin@apple.com>
 

trunk/Source/WebKit2/UIProcess/Launcher/qt/ProcessLauncherQt.cpp

@@ -76 +76 @@
 {
 #if defined Q_OS_LINUX
+#ifndef NDEBUG
+    if (getenv("QT_WEBKIT_KEEP_ALIVE_WEB_PROCESS"))
+        return;
+#endif
     prctl(PR_SET_PDEATHSIG, SIGKILL);
 #endif