Adding histograms for better Omnibox prerender performance tracking.

chrome/browser/prerender/prerender_histograms.cc

 // Copyright (c) 2011 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"
 #include "chrome/browser/prerender/prerender_util.h"
+#include "chrome/browser/prerender/prerender_field_trial.h"
 
 namespace prerender {
 
 namespace {
 
 // Time window for which we will record windowed PLT's from the last
 // observed link rel=prefetch tag.
 const int kWindowDurationSeconds = 30;
 
 std::string ComposeHistogramName(const std::string& prefix_type,
@@ skipping 118 matching lines @@
   last_experiment_id_ = experiment;
 
   // If we observe multiple tags within the 30 second window, we will still
   // reset the window to begin at the most recent occurrence, so that we will
   // always be in a window in the 30 seconds from each occurrence.
   last_prerender_seen_time_ = GetCurrentTimeTicks();
   seen_any_pageload_ = false;
   seen_pageload_started_after_prerender_ = false;
 }
 
+void PrerenderHistograms::RecordPrerenderFromOmnibox() const {
+  UMA_HISTOGRAM_COUNTS("Prerender.OmniboxPrerenderCount_" +
+                       GetOmniboxHistogramSuffix(), 1);
+}
+
+void PrerenderHistograms::RecordOmniboxUsedPrerender(Origin origin) const {

[tburkard, 2011/11/22 18:52:05]

You should use the PREFIXED_HISTOGRAM family of macros to do this, or how is
this different from any of the other ones?

+  if (origin == ORIGIN_OMNIBOX_ORIGINAL ||
+      origin == ORIGIN_OMNIBOX_CONSERVATIVE ||
+      origin == ORIGIN_OMNIBOX_EXACT ||
+      origin == ORIGIN_OMNIBOX_EXACT_FULL) {
+    UMA_HISTOGRAM_COUNTS("Prerender.OmniboxNavigationsUsedPrerenderCount_" +
+                         GetOmniboxHistogramSuffix(), 1);
+  }
+}
+
 base::TimeTicks PrerenderHistograms::GetCurrentTimeTicks() const {
   return base::TimeTicks::Now();
 }
 
 // Helper macro for histograms.
 #define RECORD_PLT(tag, perceived_page_load_time) { \
   PREFIXED_HISTOGRAM( \
     base::FieldTrial::MakeName(tag, "Prefetch"), \
     UMA_HISTOGRAM_CUSTOM_TIMES( \
         name, \
@@ skipping 111 matching lines @@
   return last_origin_;
 }
 
 bool PrerenderHistograms::IsOriginExperimentWash() const {
   if (!WithinWindow())
     return false;
   return origin_experiment_wash_;
 }
 
 }  // namespace prerender