Make a separate histogram for MatchComplete Final Status'es and

chrome/browser/prerender/prerender_histograms.cc

 // Copyright (c) 2012 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_histograms.h"
 
 #include <string>
 
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram.h"
@@ skipping 151 matching lines @@
   PREFIXED_HISTOGRAM( \
     base::FieldTrial::MakeName(tag, "Prerender"), \
     UMA_HISTOGRAM_CUSTOM_TIMES( \
         name, \
         perceived_page_load_time, \
         base::TimeDelta::FromMilliseconds(10), \
         base::TimeDelta::FromSeconds(60), \
         100)); \
 }
 
+// Summary of all histograms Perceived PLT histograms:
+// (all prefixed PerceivedPLT)
+// PerceivedPLT -- Perceived Pageloadtimes (PPLT) for all pages in the group.
+// ...Windowed -- PPLT for pages in the 30s after a prerender is created.
+// ...Matched -- A prerendered page that was swapped in.  In the NoUse
+// and Control group cases, while nothing ever gets swapped in, we do keep
+// track of what would be prerendered and would be swapped in -- and those
+// cases are what is classified as Match for these groups.
+// ...MatchedComplete -- A prerendered page that was swapped in + a few
+// that were not swapped in so that the set of pages lines up more closely with
+// the control group.
+// ...FirstAfterMiss -- First page to finish loading after a prerender, which
+// is different from the page that was prerendered.
+// ...FirstAfterMissNonOverlapping -- Same as FirstAfterMiss, but only
+// triggering for the first page to finish after the prerender that also started
+// after the prerender started.
+// ...FirstAfterMissBoth -- pages meeting
+// FirstAfterMiss AND FirstAfterMissNonOverlapping
+// ...FirstAfterMissAnyOnly -- pages meeting
+// FirstAfterMiss but NOT FirstAfterMissNonOverlapping
+// ..FirstAfterMissNonOverlappingOnly -- pages meeting
+// FirstAfterMissNonOverlapping but NOT FirstAfterMiss
+
 void PrerenderHistograms::RecordPerceivedPageLoadTime(
     base::TimeDelta perceived_page_load_time, bool was_prerender,
     bool was_complete_prerender, const GURL& url) {
   if (!IsWebURL(url))
     return;
   bool within_window = WithinWindow();
   bool is_google_url = IsGoogleDomain(url);
   RECORD_PLT("PerceivedPLT", perceived_page_load_time);
   if (within_window)
     RECORD_PLT("PerceivedPLTWindowed", perceived_page_load_time);
@@ skipping 67 matching lines @@
       UMA_HISTOGRAM_COUNTS(name, count));
 }
 
 void PrerenderHistograms::RecordTimeBetweenPrerenderRequests(
     base::TimeDelta time) const {
   PREFIXED_HISTOGRAM(
       "TimeBetweenPrerenderRequests",
       UMA_HISTOGRAM_TIMES(name, time));
 }
 
-void PrerenderHistograms::RecordFinalStatus(Origin origin,
-                                            uint8 experiment_id,
-                                            FinalStatus final_status) const {
+void PrerenderHistograms::RecordFinalStatus(
+    Origin origin,
+    uint8 experiment_id,
+    PrerenderContents::MatchCompleteStatus mc_status,
+    FinalStatus final_status) const {
   DCHECK(final_status != FINAL_STATUS_MAX);
-  PREFIXED_HISTOGRAM_ORIGIN_EXPERIMENT(
-      base::FieldTrial::MakeName("FinalStatus", "Prerender"),
-      origin, experiment_id,
-      UMA_HISTOGRAM_ENUMERATION(name, final_status, FINAL_STATUS_MAX));
+
+  // There are three cases for MatchCompleteStatus:
+  // MATCH_COMPLETE_DEFAULT:
+  // In this case, Match & MatchComplete line up.  So we record this in both
+  // histograms.
+  // MATCH_COMPLETE_REPLACED: The actual prerender was replaced by a dummy.
+  // So we only record it in (the actual) FinalStatus, but not MatchComplete.
+  // MATCH_COMPLETE_REPLACEMENT: This is a pseudo element to emulate what
+  // the control group would do.  Since it won't actually be swapped in,
+  // it may not go into FinalStatus.  Since in the control group it would be
+  // swapped in though, it must go into MatchComplete.
+
+  if (mc_status != PrerenderContents::MATCH_COMPLETE_REPLACEMENT) {
+    PREFIXED_HISTOGRAM_ORIGIN_EXPERIMENT(
+        base::FieldTrial::MakeName("FinalStatus", "Prerender"),
+        origin, experiment_id,
+        UMA_HISTOGRAM_ENUMERATION(name, final_status, FINAL_STATUS_MAX));
+  }
+  if (mc_status != PrerenderContents::MATCH_COMPLETE_REPLACED) {
+    PREFIXED_HISTOGRAM_ORIGIN_EXPERIMENT(
+        base::FieldTrial::MakeName("FinalStatusMatchComplete", "Prerender"),
+        origin, experiment_id,
+        UMA_HISTOGRAM_ENUMERATION(name, final_status, FINAL_STATUS_MAX));
+  }
 }
 
 uint8 PrerenderHistograms::GetCurrentExperimentId() const {
   if (!WithinWindow())
     return kNoExperiment;
   return last_experiment_id_;
 }
 
 Origin PrerenderHistograms::GetCurrentOrigin() const {
   if (!WithinWindow())
     return ORIGIN_LINK_REL_PRERENDER;
   return last_origin_;
 }
 
 bool PrerenderHistograms::IsOriginExperimentWash() const {
   if (!WithinWindow())
     return false;
   return origin_experiment_wash_;
 }
 
 }  // namespace prerender