Fix for Issue 3258: Tabs created when tabstrip is compressed pending resize r...

chrome/browser/tabs/tab_strip_model.cc

 // Copyright (c) 2006-2008 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 <algorithm>
 
 #include "base/gfx/point.h"
 #include "base/logging.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_about_handler.h"
@@ skipping 515 matching lines @@
 
 bool TabStripModel::InternalCloseTabContentsAt(int index,
                                                bool create_historical_tab) {
   TabContents* detached_contents = GetContentsAt(index);
 
   if (TabHasUnloadListener(index)) {
     // If the page has unload listeners, then we tell the renderer to fire
     // them. Once they have fired, we'll get a message back saying whether
     // to proceed closing the page or not, which sends us back to this method
     // with the HasUnloadListener bit cleared.
-    WebContents* web_contents = GetContentsAt(index)->AsWebContents();
+    WebContents* web_contents = detached_contents->AsWebContents();
     // If we hit this code path, the tab had better be a WebContents tab.
     DCHECK(web_contents);
     web_contents->render_view_host()->FirePageBeforeUnload();
     return false;
   }
 
   // TODO: Now that we know the tab has no unload/beforeunload listeners,
   // we should be able to do a fast shutdown of the RenderViewProcess.
   // Make sure that we actually do.
 
@@ skipping 56 matching lines @@
 }
 
 // static
 bool TabStripModel::OpenerMatches(TabContentsData* data,
                                   NavigationController* opener,
                                   bool use_group) {
   return data->opener == opener || (use_group && data->group == opener);
 }