Don't load plugins on prerendered pages until the pages are displayed....

chrome/browser/prerender/prerender_manager.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/prerender/prerender_manager.h"
 
 #include "base/logging.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/prerender/prerender_contents.h"
+#include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/browser/tab_contents/render_view_host_manager.h"
+#include "chrome/common/render_messages.h"
 
 struct PrerenderManager::PrerenderContentsData {
   PrerenderContents* contents_;
   base::Time start_time_;
   GURL url_;
   PrerenderContentsData(PrerenderContents* contents,
                         base::Time start_time,
                         GURL url)
       : contents_(contents),
         start_time_(start_time),
@@ skipping 63 matching lines @@
 }
 
 bool PrerenderManager::MaybeUsePreloadedPage(TabContents* tc, const GURL& url) {
   DCHECK(CalledOnValidThread());
   scoped_ptr<PrerenderContents> pc(GetEntry(url));
   if (pc.get() == NULL)
     return false;
 
   RenderViewHost* rvh = pc->render_view_host();
   pc->set_render_view_host(NULL);
+  rvh->Send(new ViewMsg_DisplayPrerenderedPage(rvh->routing_id()));

[cbentzel, 2011/01/20 19:46:59]

Long term it _feels_ like this should go in the PrerenderContents rather than
the PrerenderManager, but I certainly don't think that change needs to happen
now. 

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

Hmm...Might make sense to have the PrerenderContents manage the entire swap
itself instead of the PrerenderManager.

Or might make sense to move it over to TabContents::SwapInRenderViewHost(),
since the TabContents really dictates what's visible.

   tc->SwapInRenderViewHost(rvh);
 
   ViewHostMsg_FrameNavigate_Params* p = pc->navigate_params();
   if (p != NULL)
     tc->DidNavigate(rvh, *p);
 
   string16 title = pc->title();
   if (!title.empty())
     tc->UpdateTitle(rvh, pc->page_id(), UTF16ToWideHack(title));
 
@@ skipping 19 matching lines @@
 }
 
 bool PrerenderManager::IsPrerenderElementFresh(const base::Time start) const {
   base::Time now = GetCurrentTime();
   return (now - start < max_prerender_age_);
 }
 
 PrerenderContents* PrerenderManager::CreatePrerenderContents(const GURL& url) {
   return new PrerenderContents(this, profile_, url);
 }