Reset the notification ui manager even earlier, before the IO thread goes awa...

chrome/browser/browser_process_impl.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_process_impl.h"
 
 #include <map>
 
 #include "app/clipboard/clipboard.h"
 #include "app/l10n_util.h"
@@ skipping 118 matching lines @@
   // destructors can call the URLFetcher destructor, which does a
   // PostDelayedTask operation on the IO thread.  (The IO thread will handle
   // that URLFetcher operation before going away.)
   metrics_service_.reset();
   google_url_tracker_.reset();
   intranet_redirect_detector_.reset();
 
   // Need to clear profiles (download managers) before the io_thread_.
   profile_manager_.reset();
 
+  // Need to clear the desktop notification balloons before the io_thread_,
+  // since if there are any left showing we will post tasks.
+  notification_ui_manager_.reset();
+
   // Debugger must be cleaned up before IO thread and NotificationService.
   debugger_wrapper_ = NULL;
 
   if (resource_dispatcher_host_.get()) {
     // Need to tell Safe Browsing Service that the IO thread is going away
     // since it cached a pointer to it.
     if (resource_dispatcher_host()->safe_browsing_service())
       resource_dispatcher_host()->safe_browsing_service()->ShutDown();
 
     // Cancel pending requests and prevent new requests.
@@ skipping 37 matching lines @@
   resource_dispatcher_host_.reset();
 
   // Wait for the pending print jobs to finish.
   print_job_manager_->OnQuit();
   print_job_manager_.reset();
 
   // Destroy TabCloseableStateWatcher before NotificationService since the
   // former registers for notifications.
   tab_closeable_state_watcher_.reset();
 
-  // Destroy NotificationUIManager before NotificationService is shut down.
-  notification_ui_manager_.reset();
-
   // Now OK to destroy NotificationService.
   main_notification_service_.reset();
 
   // Prior to clearing local state, we want to complete tasks pending
   // on the db thread too.
   db_thread_.reset();
 
   // At this point, no render process exist and the file, io, db, and
   // webkit threads in this process have all terminated, so it's safe
   // to access local state data such as cookies, database, or local storage.
@@ skipping 556 matching lines @@
 }
 
 void BrowserProcessImpl::OnAutoupdateTimer() {
   if (CanAutorestartForUpdate()) {
     DLOG(WARNING) << "Detected update.  Restarting browser.";
     RestartPersistentInstance();
   }
 }
 
 #endif  // (defined(OS_WIN) || defined(OS_LINUX)) && !defined(OS_CHROMEOS)