Close notification windows while message loop is still active

chrome/browser/ui/browser_list_aura.cc

 // 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.
 
 #include "chrome/browser/ui/browser_list.h"
 
 #include "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/notifications/notification_ui_manager.h"
+
+#if defined(USE_ASH)
+#include "ash/shell.h"
+#endif
 
 // static
 void BrowserList::HandleAppExitingForPlatform() {
+  // Close All non browser windows now. Those includes notifications
+  // and windows created by Ash (launcher, background, etc).
+  g_browser_process->notification_ui_manager()->CancelAll();
+#if defined(USE_ASH)
+  // Shutdown the shell while the message loop is still alive.
+  // Don't delete ash shell instance yet because someone may try to
+  // access the shell, which will create new instance. We rather
+  // want to let it die if that happens.
+  ash::Shell::GetInstance()->Shutdown();

[Ben Goodger (Google), 2012/04/10 22:35:02]

can you delete the instance and then set shutdown_ statically so it knows not to
recreate, rather than leaving a non-NULL Shell instance with a bunch of NULL
ptrs hanging off it? 

[oshima, 2012/04/10 23:06:55]

Looks like GetInstance no longer creates instance. Changed to DeleteInstance()

+#endif
 #if defined(OS_CHROMEOS)
   if (!CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kDisableZeroBrowsersOpenForTests)) {
     // App is exiting, call EndKeepAlive() on behalf of Aura Shell.
     BrowserList::EndKeepAlive();
     // Make sure we have notified the session manager that we are exiting.
     // This might be called from FastShutdown() or CloseAllBrowsers(), but not
     // if something prevents a browser from closing before SetTryingToQuit()
     // gets called (e.g. browser->TabsNeedBeforeUnloadFired() is true).
     // NotifyAndTerminate does nothing if called more than once.
     BrowserList::NotifyAndTerminate(true);
   }
 #endif // OS_CHROMEOS
 }