Separate same domain and cross domain <link rel=...> prerenders for reporting.

chrome/browser/prerender/prerender_histograms.cc

Index: chrome/browser/prerender/prerender_histograms.cc
diff --git a/chrome/browser/prerender/prerender_histograms.cc b/chrome/browser/prerender/prerender_histograms.cc
index ced90b871758ec4d0b31c721d9883a288a732e27..90e923943ea0ffd8d8d763ad2cd4f0fdfb6fef0f 100644
--- a/chrome/browser/prerender/prerender_histograms.cc
+++ b/chrome/browser/prerender/prerender_histograms.cc
@@ -52,8 +52,10 @@ std::string GetHistogramName(Origin origin, uint8 experiment_id,
   switch (origin) {
     case ORIGIN_OMNIBOX:
       return ComposeHistogramName("omnibox", name);
-    case ORIGIN_LINK_REL_PRERENDER:
-      return ComposeHistogramName("web", name);
+    case ORIGIN_LINK_REL_PRERENDER_SAMEDOMAIN:
+      return ComposeHistogramName("websame", name);
+    case ORIGIN_LINK_REL_PRERENDER_CROSSDOMAIN:
+      return ComposeHistogramName("webcross", name);

[mmenke, 2012/10/17 15:12:16]

nit:  Move these after ORIGIN_NONE, to match declaration order, and the order in
PREFIXED_HISTOGRAM_INTERNAL.

[gavinp, 2012/10/17 20:57:49]

Done.

     case ORIGIN_NONE:
       return ComposeHistogramName("none", name);
     case ORIGIN_GWS_PRERENDER:  // Handled above.
@@ -105,12 +107,14 @@ bool OriginIsOmnibox(Origin origin) {
   } else if (experiment != kNoExperiment && \
              (origin != ORIGIN_GWS_PRERENDER || \
               experiment != recording_experiment)) { \
-  } else if (origin == ORIGIN_LINK_REL_PRERENDER) { \
-    HISTOGRAM; \
   } else if (origin == ORIGIN_OMNIBOX) { \
     HISTOGRAM; \
   } else if (origin == ORIGIN_NONE) { \
     HISTOGRAM; \
+  } else if (origin == ORIGIN_LINK_REL_PRERENDER_SAMEDOMAIN) { \
+    HISTOGRAM; \
+  } else if (origin == ORIGIN_LINK_REL_PRERENDER_CROSSDOMAIN) { \
+    HISTOGRAM; \
   } else if (experiment != kNoExperiment) { \
     HISTOGRAM; \
   } else { \
@@ -120,7 +124,7 @@ bool OriginIsOmnibox(Origin origin) {
 
 PrerenderHistograms::PrerenderHistograms()
     : last_experiment_id_(kNoExperiment),
-      last_origin_(ORIGIN_LINK_REL_PRERENDER),
+      last_origin_(ORIGIN_MAX),
       origin_experiment_wash_(false),
       seen_any_pageload_(true),
       seen_pageload_started_after_prerender_(true) {