Do conservative prerendering based on the LocalPredictor in Dev/Canary.

chrome/browser/prerender/prerender_tab_helper.cc

Index: chrome/browser/prerender/prerender_tab_helper.cc
===================================================================
--- chrome/browser/prerender/prerender_tab_helper.cc	(revision 198752)
+++ chrome/browser/prerender/prerender_tab_helper.cc	(working copy)
@@ -8,6 +8,7 @@
 #include "base/strings/string_number_conversions.h"
 #include "base/time.h"
 #include "chrome/browser/predictors/logged_in_predictor_table.h"
+#include "chrome/browser/prerender/prerender_field_trial.h"
 #include "chrome/browser/prerender/prerender_histograms.h"
 #include "chrome/browser/prerender/prerender_manager.h"
 #include "chrome/browser/prerender/prerender_manager_factory.h"
@@ -152,6 +153,8 @@
   if (prerender_manager->IsWebContentsPrerendering(web_contents(), NULL))
     return;
   prerender_manager->MarkWebContentsAsNotPrerendered(web_contents());
+  if (IsLocalPredictorEnabled())

[Shishir, 2013/05/07 22:57:30]

In the previous line you are marking the web_contents as not prerendering and
then trying to prerender it. Looks like the prerender manager will be
inconsistent. Also is ProvisionalChangeToMainFrameUrl the right place to do this
swap? We currently do it in chrome::Navigate.

[tburkard, 2013/05/07 23:21:59]

No, this only means that the data currently in the webcontents was not
prerendered. This is done because the URL changed. If we swap in, we will update
this to say that it was prerendered, based on the swap-in. So this is correct
(in particular, because of the order).

+    prerender_manager->MaybeUsePrerenderedPage(web_contents(), url);
 }
 
 void PrerenderTabHelper::DidCommitProvisionalLoadForFrame(
@@ -172,6 +175,8 @@
   if (prerender_manager->IsWebContentsPrerendering(web_contents(), NULL))
     return;
   prerender_manager->RecordNavigation(validated_url);
+  if (IsLocalPredictorEnabled())

[Shishir, 2013/05/07 22:57:30]

Why are we trying to swap at both start and commit of provisional load? Isn't
just one enough?

[tburkard, 2013/05/07 23:21:59]

Some URL changes only show up in one or the other, but not in both, which is why
we need this twice.

[Shishir, 2013/05/08 20:19:20]

Can you mention which ones show up on in only one or other? Isn't the commit of
a load a bit late to swap in prerendering. It possibly means that we have
recieved a response for the request. 

[tburkard, 2013/05/08 20:35:47]

Removing these changes in this file altogether, since Chris is preparing a new
change to address this that is cleaner.

+    prerender_manager->MaybeUsePrerenderedPage(web_contents(), validated_url);
 }
 
 void PrerenderTabHelper::DidStopLoading(