Once we initiate shutdown sequence, don't show crash bar on subsequent startu...

chrome/browser/ui/browser_list.cc

 // Copyright (c) 2010 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/browser_list.h"
 
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "build/build_config.h"
@@ skipping 165 matching lines @@
       NotificationService::NoDetails());
 
   // Send out notifications after add has occurred. Do some basic checking to
   // try to catch evil observers that change the list from under us.
   size_t original_count = observers_.size();
   FOR_EACH_OBSERVER(Observer, observers_, OnBrowserAdded(browser));
   DCHECK_EQ(original_count, observers_.size())
       << "observer list modified during notification";
 }
 
+// static
+void BrowserList::MarkAsCleanShutdown() {
+  for (const_iterator i = begin(); i != end(); ++i) {
+    (*i)->profile()->MarkAsCleanShutdown();
+  }
+}
 
 // static
 void BrowserList::NotifyAndTerminate() {
   NotificationService::current()->Notify(NotificationType::APP_TERMINATING,
                                          NotificationService::AllSources(),
                                          NotificationService::NoDetails());
 #if defined(OS_CHROMEOS)
   chromeos::WmIpc::instance()->NotifyAboutSignout();
   if (chromeos::CrosLibrary::Get()->EnsureLoaded()) {
     chromeos::CrosLibrary::Get()->GetLoginLibrary()->StopSession("");
@@ skipping 59 matching lines @@
 
 // static
 void BrowserList::RemoveObserver(BrowserList::Observer* observer) {
   observers_.RemoveObserver(observer);
 }
 
 #if defined(OS_CHROMEOS)
 // static
 bool BrowserList::NeedBeforeUnloadFired() {
   bool need_before_unload_fired = false;
-  for (BrowserList::const_iterator i = BrowserList::begin();
-       i != BrowserList::end(); ++i) {
+  for (const_iterator i = begin(); i != end(); ++i) {
     need_before_unload_fired = need_before_unload_fired ||
       (*i)->TabsNeedBeforeUnloadFired();
   }
   return need_before_unload_fired;
 }
 
 // static
 bool BrowserList::PendingDownloads() {
-  for (BrowserList::const_iterator i = BrowserList::begin();
-       i != BrowserList::end(); ++i) {
+  for (const_iterator i = begin(); i != end(); ++i) {
     bool normal_downloads_are_present = false;
     bool incognito_downloads_are_present = false;
     (*i)->CheckDownloadsInProgress(&normal_downloads_are_present,
                                    &incognito_downloads_are_present);
     if (normal_downloads_are_present || incognito_downloads_are_present)
       return true;
   }
   return false;
 }
 #endif
@@ skipping 60 matching lines @@
       && !PendingDownloads()) {
     NotifyAndTerminate();
     return;
   }
 #endif
   CloseAllBrowsersAndExit();
 }
 
 // static
 void BrowserList::CloseAllBrowsersAndExit() {
+  MarkAsCleanShutdown(); // Don't notify users of crashes beyond this point.

[Daniel Erat, 2011/01/25 02:29:42]

I think you might be missing the fast-shutdown path by putting this here; see
BrowserList::Exit().  NotifyAndTerminate() seemed to be a better choice for the
notifying-wm-about-signout stuff that I was working on (it also gets called by
CloseAllBrowsers(), as far as I can see).

   NotificationService::current()->Notify(
       NotificationType::APP_EXITING,
       NotificationService::AllSources(),
       NotificationService::NoDetails());
 
 #if !defined(OS_MACOSX)
   // On most platforms, closing all windows causes the application to exit.
   CloseAllBrowsers();
 #else
   // On the Mac, the application continues to run once all windows are closed.
@@ skipping 226 matching lines @@
     }
 
     TabContents* next_tab =
         (*browser_iterator_)->GetTabContentsAt(web_view_index_);
     if (next_tab) {
       cur_ = next_tab;
       return;
     }
   }
 }