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 "ash/shell.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 30 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"
+#include "chrome/browser/browser_process_platform_part_winaura.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, whose host holds a reference to
+    // g_browser_process, needs to be killed.
+    g_browser_process->platform_part()->TerminateMetroViewerProcess();

[sky, 2013/05/15 21:13:18]

How about naming this AttemptExit so that you can put it in the base class and
remove the ifdefs here?

[gab, 2013/05/17 23:19:18]

Done on top of https://codereview.chromium.org/14951007.

+  }
+#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 282 matching lines @@
   //    environment is still active.
   if (browser->host_desktop_type() == chrome::HOST_DESKTOP_TYPE_NATIVE)
     return !ash::Shell::HasInstance();
   else if (browser->host_desktop_type() == chrome::HOST_DESKTOP_TYPE_ASH)
     return BrowserList::GetInstance(chrome::HOST_DESKTOP_TYPE_NATIVE)->empty();
 #endif
   return true;
 }
 
 }  // namespace chrome