Prevent Chrome from quitting when apps are open. (Mac)

apps/app_window_registry.cc

 // Copyright 2014 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.
 
 #include "apps/app_window.h"
 #include "apps/app_window_registry.h"
 #include "apps/apps_client.h"
 #include "apps/ui/native_app_window.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "content/public/browser/browser_context.h"
@@ skipping 210 matching lines @@
 
     for (const_iterator j = app_windows.begin(); j != app_windows.end(); ++j) {
       if ((*j)->window_type() & window_type_mask)
         return true;
     }
   }
 
   return false;
 }
 
+// static
+void AppWindowRegistry::CloseAllAppWindows() {
+  std::vector<content::BrowserContext*> contexts =
+      AppsClient::Get()->GetLoadedBrowserContexts();
+  for (std::vector<content::BrowserContext*>::const_iterator i =
+           contexts.begin();
+       i != contexts.end();
+       ++i) {
+    AppWindowRegistry* registry =
+        Factory::GetForBrowserContext(*i, false /* create */);
+    if (!registry)
+      continue;
+
+    const AppWindowList& app_windows = registry->app_windows();
+    while (!app_windows.empty())

[tapted, 2014/04/03 03:49:15]

nit: I would just use !registry->app_windows_.empty() (and
registry->app_windows_.front() below) - I don't think there are any style rules
against accessing private members like this from a static function.

[I'm still scared by that const-reference ;) ]

[jackhou1, 2014/04/03 04:00:52]

Done.

+      app_windows.front()->GetBaseWindow()->Close();
+  }
+}
+
 void AppWindowRegistry::OnDevToolsStateChanged(
     content::DevToolsAgentHost* agent_host,
     bool attached) {
   content::RenderViewHost* rvh = agent_host->GetRenderViewHost();
   // Ignore unrelated notifications.
   if (!rvh ||
       rvh->GetSiteInstance()->GetProcess()->GetBrowserContext() != context_)
     return;
 
   std::string key = GetWindowKeyForRenderViewHost(this, rvh);
@@ skipping 57 matching lines @@
   return false;
 }
 
 content::BrowserContext* AppWindowRegistry::Factory::GetBrowserContextToUse(
     content::BrowserContext* context) const {
   return extensions::ExtensionsBrowserClient::Get()->GetOriginalContext(
       context);
 }
 
 }  // namespace extensions