Leave apps running on Windows and Linux when quitting Chrome from the wrench menu.

chrome/browser/app_controller_mac.mm

Index: chrome/browser/app_controller_mac.mm
diff --git a/chrome/browser/app_controller_mac.mm b/chrome/browser/app_controller_mac.mm
index 34736e6251be73809fade4cc03caf67b8cdb177b..f2832dd5b669a6bf7027fc59c1c1d762118355f0 100644
--- a/chrome/browser/app_controller_mac.mm
+++ b/chrome/browser/app_controller_mac.mm
@@ -359,15 +359,12 @@ class AppControllerProfileObserver : public ProfileInfoCacheObserver {
 
   size_t num_browsers = chrome::GetTotalBrowserCount();
 
-  // Initiate a shutdown (via chrome::CloseAllBrowsers()) if we aren't
-  // already shutting down.
-  if (!browser_shutdown::IsTryingToQuit()) {

[benwells, 2013/10/21 05:57:22]

Why was this check here? Or put another way, why have you removed it?

[Sam McNally, 2013/10/21 06:45:35]

Previously, CloseAllBrowsers called SetTryingToQuit. Now, the callers need to
call SetTryingToQuit so giving up if we're already trying to quit would
quitting.

[benwells, 2013/10/21 21:12:51]

The check seems odd. I'm worried there an obscure case where this code path
could be hit while quitting, and needed to be prevented. Could you look up when
the if clause was added and see if it was in response to a bug?

[Sam McNally, 2013/10/22 00:16:27]

Done.

-    content::NotificationService::current()->Notify(
-        chrome::NOTIFICATION_CLOSE_ALL_BROWSERS_REQUEST,
-        content::NotificationService::AllSources(),
-        content::NotificationService::NoDetails());
-    chrome::CloseAllBrowsers();
-  }
+  // Initiate a shutdown (via chrome::CloseAllBrowsersAndQuit()).
+  content::NotificationService::current()->Notify(
+      chrome::NOTIFICATION_CLOSE_ALL_BROWSERS_REQUEST,
+      content::NotificationService::AllSources(),
+      content::NotificationService::NoDetails());
+  chrome::CloseAllBrowsersAndQuit();
 
   return num_browsers == 0 ? YES : NO;
 }