Adding more histograms to ResourcePrefetchPredictor.

chrome/browser/prerender/prerender_field_trial.cc

Index: chrome/browser/prerender/prerender_field_trial.cc
diff --git a/chrome/browser/prerender/prerender_field_trial.cc b/chrome/browser/prerender/prerender_field_trial.cc
index e47c412b9fa8487351c2f0160e8c300fa53e90e5..06ab1fd081d608ade95c131696431f89c6128843 100644
--- a/chrome/browser/prerender/prerender_field_trial.cc
+++ b/chrome/browser/prerender/prerender_field_trial.cc
@@ -274,10 +274,7 @@ void ConfigureSpeculativePrefetching() {
       kDisabledProbability);
 }
 
-bool IsSpeculativeResourcePrefetchingLearningEnabled(Profile* profile) {
-  if (!profile)

[dominich, 2012/08/30 18:25:23]

this might mean that tests with NULL profiles will be entered into a field
trial. I'm not sure if the FieldTrial code already checks that for tests, or if
you're manually setting the commandline switch for all tests to disabled, but
it's something to be aware of.

[Shishir, 2012/08/30 18:48:09]

Digging a little deeper, I need to disable this when the profile is off the
record. This could be the reason I am seeing so many empty databases.  So I
think I will have to put this profile check back in.

-    return false;
-
+bool IsSpeculativeResourcePrefetchingLearningEnabled() {
   // Override any field trial groups if the user has set a command line flag.
   if (CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kSpeculativeResourcePrefetching)) {