Don't close crashed tabs from extensions with background pages.

content/browser/tab_contents/render_view_host_manager.cc

 // Copyright (c) 2011 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 "content/browser/tab_contents/render_view_host_manager.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/renderer_host/render_view_host_delegate.h"
@@ skipping 78 matching lines @@
   // its first page.  (Bug 1145340)
   if (dest_render_view_host != render_view_host_ &&
       !render_view_host_->IsRenderViewLive()) {
     // Note: we don't call InitRenderView here because we are navigating away
     // soon anyway, and we don't have the NavigationEntry for this host.
     delegate_->CreateRenderViewForRenderManager(render_view_host_);
   }
 
   // If the renderer crashed, then try to create a new one to satisfy this
   // navigation request.
+  // TODO(yoz): Should we try to reload extensions here instead??

[Matt Perry, 2011/11/10 22:07:59]

content/ shouldn't know about extensions. if we want to reload them at this
point, it would have to be in the delegate. just remove this TODO.

[Yoyo Zhou, 2011/11/10 23:26:30]

Yeah, this was from an earlier approach.

   if (!dest_render_view_host->IsRenderViewLive()) {
     if (!InitRenderView(dest_render_view_host, entry))
       return NULL;
 
     // Now that we've created a new renderer, be sure to hide it if it isn't
     // our primary one.  Otherwise, we might crash if we try to call Show()
     // on it later.
     if (dest_render_view_host != render_view_host_ &&
         dest_render_view_host->view()) {
       dest_render_view_host->view()->Hide();
@@ skipping 687 matching lines @@
   }
 }
 
 bool RenderViewHostManager::IsSwappedOut(RenderViewHost* rvh) {
   if (!rvh->site_instance())
     return false;
 
   return swapped_out_hosts_.find(rvh->site_instance()->id()) !=
       swapped_out_hosts_.end();
 }