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

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

Issue 11551003: Change multi-prerender API to include per launcher slots. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: clear to land Created 8 years 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
« no previous file with comments | « chrome/browser/prerender/prerender_histograms.h ('k') | chrome/browser/prerender/prerender_link_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_histograms.cc
diff --git a/chrome/browser/prerender/prerender_histograms.cc b/chrome/browser/prerender/prerender_histograms.cc
index 7395842983b118ce103b84a153af916f7041ffe3..73061c395dd21665aa1d236da267e11f9295b53b 100644
--- a/chrome/browser/prerender/prerender_histograms.cc
+++ b/chrome/browser/prerender/prerender_histograms.cc
@@ -167,16 +167,13 @@ void PrerenderHistograms::RecordPrerenderStarted(Origin origin) const {
}
}
-void PrerenderHistograms::RecordConcurrency(size_t prerender_count,
- size_t max_concurrency) const {
- static const size_t kMaxRecordableConcurrency = 3;
- DCHECK_GE(kMaxRecordableConcurrency, max_concurrency);
- if (max_concurrency > 1) {
- UMA_HISTOGRAM_ENUMERATION(
- StringPrintf("Prerender.PrerenderCountOf%" PRIuS "Max",
- max_concurrency),
- prerender_count, kMaxRecordableConcurrency + 1);
- }
+void PrerenderHistograms::RecordConcurrency(size_t prerender_count) const {
+ static const size_t kMaxRecordableConcurrency = 20;
+ DCHECK_GE(kMaxRecordableConcurrency, Config().max_link_concurrency);
+ UMA_HISTOGRAM_ENUMERATION(
+ StringPrintf("Prerender.PrerenderCountOf%" PRIuS "Max",
+ kMaxRecordableConcurrency),
+ prerender_count, kMaxRecordableConcurrency + 1);
}
void PrerenderHistograms::RecordUsedPrerender(Origin origin) const {
« no previous file with comments | « chrome/browser/prerender/prerender_histograms.h ('k') | chrome/browser/prerender/prerender_link_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698