Removes phantom tabs.

chrome/browser/extensions/extension_tabs_module.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/extensions/extension_tabs_module.h"
 
 #include "base/base64.h"
 #include "base/string_util.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_list.h"
@@ skipping 766 matching lines @@
 bool RemoveTabFunction::RunImpl() {
   int tab_id;
   EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(0, &tab_id));
 
   Browser* browser = NULL;
   TabContents* contents = NULL;
   if (!GetTabById(tab_id, profile(), include_incognito(),
                   &browser, NULL, &contents, NULL, &error_))
     return false;
 
-  int tab_index = browser->GetIndexOfController(&contents->controller());
-  if (browser->tabstrip_model()->IsPhantomTab(tab_index)) {
-    // Don't allow closing phantom tabs.
-    error_ = keys::kCannotRemovePhantomTab;
-    return false;
-  }
-
   // Close the tab in this convoluted way, since there's a chance that the tab
   // is being dragged, or we're in some other nested event loop. This code path
   // should ensure that the tab is safely closed under such circumstances,
   // whereas |Browser::CloseTabContents()| does not.
   RenderViewHost* render_view_host = contents->render_view_host();
   render_view_host->delegate()->Close(render_view_host);
   return true;
 }
 
 bool CaptureVisibleTabFunction::RunImpl() {
@@ skipping 174 matching lines @@
   } else {
     browser = GetCurrentBrowser();
     if (!browser)
       return false;
     contents = browser->tabstrip_model()->GetSelectedTabContents();
     if (!contents)
       return false;
   }
 
   if (contents->controller().needs_reload()) {
-    // If the tab hasn't been loaded, such as happens with phantom tabs, don't
-    // wait for the tab to load, instead return.
+    // If the tab hasn't been loaded don't wait for the tab to load, instead
+    // return.
     error_ = keys::kCannotDetermineLanguageOfUnloadedTab;
     return false;
   }
 
   AddRef();  // Balanced in GotLanguage()
 
   if (!contents->language_state().original_language().empty()) {
     // Delay the callback invocation until after the current JS call has
     // returned.
     MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
@@ skipping 86 matching lines @@
 }
 
 static GURL ResolvePossiblyRelativeURL(std::string url_string,
                                        Extension* extension) {
   GURL url = GURL(url_string);
   if (!url.is_valid())
     url = extension->GetResourceURL(url_string);
 
   return url;
 }