In WinAura, also kill the Metro viewer process in AttemptExit().

chrome/browser/lifetime/application_lifetime.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/lifetime/application_lifetime.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
@@ skipping 29 matching lines @@
 
 #if defined(OS_CHROMEOS)
 #include "base/chromeos/chromeos_version.h"
 #include "chrome/browser/chromeos/boot_times_loader.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/session_manager_client.h"
 #include "chromeos/dbus/update_engine_client.h"
 #endif
 
+#if defined(OS_WIN) && defined(USE_AURA)
+#include "base/win/windows_version.h"
+#endif
+
 namespace chrome {
 namespace {
 
 // Returns true if all browsers can be closed without user interaction.
 // This currently checks if there is pending download, or if it needs to
 // handle unload handler.
 bool AreAllBrowsersCloseable() {
   chrome::BrowserIterator browser_it;
   if (browser_it.done())
     return true;
@@ skipping 35 matching lines @@
   // Tell the Java code to finish() the Activity.
   TerminateAndroid();
 #elif defined(OS_MACOSX)
   // On the Mac, the application continues to run once all windows are closed.
   // Terminate will result in a CloseAllBrowsers() call, and once (and if)
   // that is done, will cause the application to exit cleanly.
   chrome_browser_application_mac::Terminate();
 #else
   // On most platforms, closing all windows causes the application to exit.
   CloseAllBrowsers();
+#if defined(OS_WIN) && defined(USE_AURA)
+  if (base::win::GetVersion() >= base::win::VERSION_WIN8) {
+    // On Windows 8, Ash might be active in Metro and won't go away even if all
+    // browsers are closed. The viewer process, which holds a reference to

[robertshield, 2013/05/10 16:51:48]

technically it's not the viewer process that holds a reference, it's the host
side of the browser-viewer connection. 

[gab, 2013/05/13 17:46:16]

Good point, clarified.

+    // g_browser_process, needs to be killed.
+    g_browser_process->TerminateMetroViewerProcess();
+  }
+#endif  // defined(OS_WIN) && defined(USE_AURA)
 #endif
 }
 
 void CloseAllBrowsers() {
   bool session_ending =
       browser_shutdown::GetShutdownType() == browser_shutdown::END_SESSION;
   // Tell everyone that we are shutting down.
   browser_shutdown::SetTryingToQuit(true);
 
 #if defined(ENABLE_SESSION_SERVICE)
@@ skipping 279 matching lines @@
   chrome::HostDesktopType other_desktop =
       (browser->host_desktop_type() == chrome::HOST_DESKTOP_TYPE_NATIVE ?
           chrome::HOST_DESKTOP_TYPE_ASH : chrome::HOST_DESKTOP_TYPE_NATIVE);
   if (!BrowserList::GetInstance(other_desktop)->empty())
     return false;
 #endif
   return true;
 }
 
 }  // namespace chrome