In Chrome ASH on Windows 8 make sure that the browser object is closed before destroying it.

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/ui/browser_list.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browser_shutdown.h"
 #include "chrome/browser/download/download_service.h"
 #include "chrome/browser/metrics/thread_watcher.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_list_impl.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/tab_contents/tab_contents.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/browser_shutdown.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/navigation_details.h"
@@ skipping 170 matching lines @@
       if (i != BrowserList::end() && browser == *i) {
         // Destroying the browser should have removed it from the browser list.
         // We should never get here.
         NOTREACHED();
         return;
       }
     }
   }
 }
 
+void CloseAllBrowsersOfType(chrome::HostDesktopType type) {

[sky, 2012/11/29 20:28:03]

I'm worried that keeping this here means other folks might call it when this
isn't meant for public consumption. Can you move it to
metro_viewer_process_host_win and name it something like
DeleteTabsInBrowserOfType to make it clear what's going on.

[ananta, 2012/11/29 21:55:42]

Done. Added a local function CloseOpenAshBrowsers to
metro_viewer_process_host_win to do this.

+  chrome::BrowserListImpl* browser_list =
+      chrome::BrowserListImpl::GetInstance(type);
+  if (browser_list) {
+    for (chrome::BrowserListImpl::const_iterator i = browser_list->begin();
+            i != browser_list->end(); ++i) {
+      Browser* browser = *i;
+      browser->window()->Close();
+      // If the attempt to Close the browser fails due to unload handlers on
+      // the page or in progress downloads, etc, destroy all tabs on the page.
+      while (browser->tab_count())
+        delete browser->tab_strip_model()->GetTabContentsAt(0);
+    }
+  }
+}
+
 void AttemptUserExit() {
 #if defined(OS_CHROMEOS)
   chromeos::BootTimesLoader::Get()->AddLogoutTimeMarker("LogoutStarted", false);
   // Write /tmp/uptime-logout-started as well.
   const char kLogoutStarted[] = "logout-started";
   chromeos::BootTimesLoader::Get()->RecordCurrentStats(kLogoutStarted);
 
   // Login screen should show up in owner's locale.
   PrefService* state = g_browser_process->local_state();
   if (state) {
@@ skipping 144 matching lines @@
         MessageLoop::current())
       CloseAllBrowsers();
   }
 }
 
 bool WillKeepAlive() {
   return g_keep_alive_count > 0;
 }
 
 }  // namespace browser