Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(173)

Unified Diff: chrome/browser/prerender/prerender_histograms.cc

Issue 10802024: Make three simultanious prerenders the default maximum in Canary and Dev only. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add an histogram Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/prerender/prerender_histograms.cc
diff --git a/chrome/browser/prerender/prerender_histograms.cc b/chrome/browser/prerender/prerender_histograms.cc
index 415b19bace613c8fb5ddd23a73e1a7e592a1af80..b6abfcaa6442ccb6a6d7b86bedda1936582032b1 100644
--- a/chrome/browser/prerender/prerender_histograms.cc
+++ b/chrome/browser/prerender/prerender_histograms.cc
@@ -148,7 +148,10 @@ void PrerenderHistograms::RecordPrerender(Origin origin, const GURL& url) {
seen_pageload_started_after_prerender_ = false;
}
-void PrerenderHistograms::RecordPrerenderStarted(Origin origin) const {
+void PrerenderHistograms::RecordPrerenderStarted(Origin origin,
+ size_t prerender_count) const {
+ UMA_HISTOGRAM_ENUMERATION("Prerender.PrerenderCount", prerender_count,
+ Config::kDefaultMaxConcurrency + 1);
dominich 2012/07/19 16:04:38 this will be a measure of how often we prerender 1
gavinp 2012/07/23 18:35:21 Good idea: I now only emit this if the max concurr
if (OriginIsOmnibox(origin)) {
UMA_HISTOGRAM_COUNTS(
StringPrintf("Prerender.OmniboxPrerenderCount%s",

Powered by Google App Engine
This is Rietveld 408576698