Consolidate Browser Creation.

chrome/browser/ui/browser_list_impl.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_impl.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browser_shutdown.h"
@@ skipping 77 matching lines @@
   bool closing_app = (browsers_.size() == 1);
 #endif  // OS_CHROMEOS
 
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_BROWSER_CLOSED,
       content::Source<Browser>(browser),
       content::Details<bool>(&closing_app));
 
   RemoveBrowserFrom(browser, &browsers_);
 
-  // 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(BrowserListObserver, observers_, OnBrowserRemoved(browser));
-  DCHECK_EQ(original_count, observers_.size())
-      << "observer list modified during notification";
 
   g_browser_process->ReleaseModule();
 
   // If we're exiting, send out the APP_TERMINATING notification to allow other
   // modules to shut themselves down.
   if (browsers_.empty() &&
       (browser_shutdown::IsTryingToQuit() ||
        g_browser_process->IsShuttingDown())) {
     // Last browser has just closed, and this is a user-initiated quit or there
     // is no module keeping the app alive, so send out our notification. No need
@@ skipping 83 matching lines @@
 
 void BrowserListImpl::RemoveBrowserFrom(Browser* browser,
                                         BrowserVector* browser_list) {
   const iterator remove_browser =
       std::find(browser_list->begin(), browser_list->end(), browser);
   if (remove_browser != browser_list->end())
     browser_list->erase(remove_browser);
 }
 
 }  // namespace chrome