Unbreak tabs.onRemove extension API in face of fast tab closure

chrome/browser/ui/browser.cc

 // Copyright 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.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // defined(OS_WIN)
@@ skipping 964 matching lines @@
     session_service->TabClosing(contents);
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_TAB_CLOSING,
       content::Source<NavigationController>(&contents->GetController()),
       content::NotificationService::NoDetails());
 
   // Sever the WebContents' connection back to us.
   SetAsDelegate(contents, NULL);
 }
 
-void Browser::TabDetachedAt(WebContents* contents, int index) {
+void Browser::TabDetachedAt(WebContents* contents,
+                            int index,
+                            bool closing_all) {
   // TabDetachedAt is called before TabStripModel has updated the
   // active index.
   int old_active_index = tab_strip_model_->active_index();
-  if (index < old_active_index && !tab_strip_model_->closing_all()) {
+  if (index < old_active_index && !closing_all) {
     SessionService* session_service =
         SessionServiceFactory::GetForProfileIfExisting(profile_);
     if (session_service)
       session_service->SetSelectedTabInWindow(session_id(),
                                               old_active_index - 1);
   }
   TabDetachedAtImpl(contents, index, DETACH_TYPE_DETACH);
 }
 
 void Browser::TabDeactivated(WebContents* contents) {
@@ skipping 1212 matching lines @@
   if (contents && !allow_js_access) {
     contents->web_contents()->GetController().LoadURL(
         target_url,
         content::Referrer(),
         content::PAGE_TRANSITION_LINK,
         std::string());  // No extra headers.
   }
 
   return contents != NULL;
 }