Cancels prerendering on prerendering RenderView crash

chrome/browser/prerender/prerender_contents.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 "chrome/browser/prerender/prerender_contents.h"
 
 #include "base/process_util.h"
 #include "base/task.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/background_contents_service.h"
@@ skipping 228 matching lines @@
 }
 
 ViewType::Type PrerenderContents::GetRenderViewType() const {
   return ViewType::BACKGROUND_CONTENTS;
 }
 
 int PrerenderContents::GetBrowserWindowID() const {
   return extension_misc::kUnknownWindowId;
 }
 
+void PrerenderContents::RenderViewGone(RenderViewHost* render_view_host,
+                                       base::TerminationStatus status,
+                                       int error_code) {
+  // Cancel the prerender if the RenderView crashes.

[cbentzel, 2011/04/22 19:55:10]

I noticed that TabContents::RenderViewGone makes sure that the render_view_host
argument matches render_view_host_ - should this? 

[mmenke, 2011/04/22 20:09:20]

The TabContents can be the delegate for multiple RenderViews - the one currently
in use, and the pending one, for navigations that can't use the active one (Like
going from chrome://blah to http://TheScaryRemoteWilderness).  We don't have to
worry about that.

+  Destroy(FINAL_STATUS_RENDERER_CRASHED);
+}
+
 void PrerenderContents::DidNavigate(
     RenderViewHost* render_view_host,
     const ViewHostMsg_FrameNavigate_Params& params) {
   // We only care when the outer frame changes.
   if (!PageTransition::IsMainFrame(params.transition))
     return;
 
   // Store the navigation params.
   ViewHostMsg_FrameNavigate_Params* p = new ViewHostMsg_FrameNavigate_Params();
   *p = params;
@@ skipping 297 matching lines @@
     return;
 
   size_t private_bytes, shared_bytes;
   if (metrics->GetMemoryBytes(&private_bytes, &shared_bytes)) {
     if (private_bytes > kMaxPrerenderPrivateMB * 1024 * 1024)
       Destroy(FINAL_STATUS_MEMORY_LIMIT_EXCEEDED);
   }
 }
 
 }  // namespace prerender