Adding histograms for better Omnibox prerender performance tracking.

chrome/browser/prerender/prerender_manager.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_manager.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 339 matching lines @@
       break;
 
     case OMNIBOX_HEURISTIC_EXACT_FULL:
       origin = ORIGIN_OMNIBOX_EXACT_FULL;
       break;
 
     default:
       NOTREACHED();
       break;
   };
+  UMA_HISTOGRAM_COUNTS("Prerender.OmniboxPrerenderCount_" +
+                       prerender::GetOmniboxHistogramSuffix(), 1);
 
   return AddPrerender(origin, std::make_pair(-1, -1), url, GURL(),
                       session_storage_namespace);
 }
 
 bool PrerenderManager::AddPrerender(
     Origin origin,
     const std::pair<int, int>& child_route_id_pair,
     const GURL& url_arg,
     const GURL& referrer,
@@ skipping 269 matching lines @@
 
   if (!prerender_contents->load_start_time().is_null()) {
     histograms_->RecordTimeUntilUsed(GetCurrentTimeTicks() -
                                      prerender_contents->load_start_time(),
                                      config_.max_age);
   }
 
   histograms_->RecordPerSessionCount(++prerenders_per_session_count_);
   prerender_contents->set_final_status(FINAL_STATUS_USED);
 
+  if (prerender_contents->origin() == ORIGIN_OMNIBOX_ORIGINAL ||
+      prerender_contents->origin() == ORIGIN_OMNIBOX_CONSERVATIVE ||
+      prerender_contents->origin() == ORIGIN_OMNIBOX_EXACT) {
+    UMA_HISTOGRAM_COUNTS("Prerender.OmniboxNavigationsUsedPrerenderCount_" +

[tburkard, 2011/11/21 23:45:39]

Wouldn't this fit nicer into prerender_histograms which already has a way to
construct such histogram names for the different prerender types?

+                         prerender::GetOmniboxHistogramSuffix(), 1);
+  }
+
   new_render_view_host->Send(
       new ChromeViewMsg_SetIsPrerendering(new_render_view_host->routing_id(),
                                           false));
 
   TabContentsWrapper* new_tab_contents =
       prerender_contents->ReleasePrerenderContents();
   TabContentsWrapper* old_tab_contents =
       TabContentsWrapper::GetCurrentWrapperForContents(tab_contents);
   DCHECK(new_tab_contents);
   DCHECK(old_tab_contents);
@@ skipping 491 matching lines @@
   if (!render_process_host || !render_process_host->GetBrowserContext())
     return NULL;
   Profile* profile = Profile::FromBrowserContext(
       render_process_host->GetBrowserContext());
   if (!profile)
     return NULL;
   return PrerenderManagerFactory::GetInstance()->GetForProfile(profile);
 }
 
 }  // namespace prerender