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

chrome/browser/app_controller_mac.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/app_controller_mac.h"
 
 #include "apps/app_shim/app_shim_mac.h"
 #include "apps/app_shim/extension_app_shim_handler_mac.h"
 #include "apps/shell_window_registry.h"
 #include "base/auto_reset.h"
@@ skipping 341 matching lines @@
   // though.) http://crbug.com/40861
 
   // Check if the user really wants to quit by employing the confirm-to-quit
   // mechanism.
   if (!browser_shutdown::IsTryingToQuit() &&
       [self applicationShouldTerminate:app] != NSTerminateNow)
     return NO;
 
   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;
 }
 
 - (void)stopTryingToTerminateApplication:(NSApplication*)app {
   if (browser_shutdown::IsTryingToQuit()) {
     // Reset the "trying to quit" state, so that closing all browser windows
     // will no longer lead to termination.
     browser_shutdown::SetTryingToQuit(false);
 
@@ skipping 1041 matching lines @@
 
 //---------------------------------------------------------------------------
 
 namespace app_controller_mac {
 
 bool IsOpeningNewWindow() {
   return g_is_opening_new_window;
 }
 
 }  // namespace app_controller_mac