Web process always pauses on launch
authorjeffm@apple.com <jeffm@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Jun 2011 22:11:19 +0000 (22:11 +0000)
committerjeffm@apple.com <jeffm@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Jun 2011 22:11:19 +0000 (22:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=63487

Reviewed by Oliver Hunt.

In the absence of the WEBKIT2_PAUSE_WEB_PROCESS_ON_LAUNCH environment variable, only pause the web process on
launch if the shift, control, and alt keys are all held down. This regressed in r89865, which always caused the
web process to pause if the shift and alt keys were not held down.

* WebProcess/WebKitMain.cpp:
(pauseProcessIfNeeded): Add some parentheses.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@89865 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/WebProcess/WebKitMain.cpp

index da65cd2..39b1ec6 100644 (file)
@@ -1,3 +1,17 @@
+2011-06-27  Jeff Miller  <jeffm@apple.com>
+
+        Reviewed by Oliver Hunt.
+
+        Web process always pauses on launch
+        https://bugs.webkit.org/show_bug.cgi?id=63487
+        
+        In the absence of the WEBKIT2_PAUSE_WEB_PROCESS_ON_LAUNCH environment variable, only pause the web process on
+        launch if the shift, control, and alt keys are all held down. This regressed in r89865, which always caused the
+        web process to pause if the shift and alt keys were not held down.
+
+        * WebProcess/WebKitMain.cpp:
+        (pauseProcessIfNeeded): Add some parentheses.
+
 2011-06-27  Huang Dongsung  <luxtella@company100.net>
 
         Reviewed by Kenneth Rohde Christiansen.
index 02bc6e0..03a8cb5 100644 (file)
@@ -150,7 +150,7 @@ static void pauseProcessIfNeeded(HMODULE module)
     // Show an alert when Ctrl-Alt-Shift is held down during launch to give the user time to attach a
     // debugger. This is useful for debugging problems that happen early in the web process's lifetime.
     const unsigned short highBitMaskShort = 0x8000;
-    if (!getenv("WEBKIT2_PAUSE_WEB_PROCESS_ON_LAUNCH") && !(::GetKeyState(VK_CONTROL) & highBitMaskShort) && (::GetKeyState(VK_MENU) & highBitMaskShort) && (::GetKeyState(VK_SHIFT) & highBitMaskShort))
+    if (!getenv("WEBKIT2_PAUSE_WEB_PROCESS_ON_LAUNCH") && !((::GetKeyState(VK_CONTROL) & highBitMaskShort) && (::GetKeyState(VK_MENU) & highBitMaskShort) && (::GetKeyState(VK_SHIFT) & highBitMaskShort)))
         return;
 
     wchar_t path[MAX_PATH];