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

Side by Side 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 7 years, 12 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/prerender/prerender_histograms.h" 5 #include "chrome/browser/prerender/prerender_histograms.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/format_macros.h" 9 #include "base/format_macros.h"
10 #include "base/metrics/field_trial.h" 10 #include "base/metrics/field_trial.h"
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 } 160 }
161 161
162 void PrerenderHistograms::RecordPrerenderStarted(Origin origin) const { 162 void PrerenderHistograms::RecordPrerenderStarted(Origin origin) const {
163 if (OriginIsOmnibox(origin)) { 163 if (OriginIsOmnibox(origin)) {
164 UMA_HISTOGRAM_ENUMERATION( 164 UMA_HISTOGRAM_ENUMERATION(
165 StringPrintf("Prerender.OmniboxPrerenderCount%s", 165 StringPrintf("Prerender.OmniboxPrerenderCount%s",
166 PrerenderManager::GetModeString()), 1, 2); 166 PrerenderManager::GetModeString()), 1, 2);
167 } 167 }
168 } 168 }
169 169
170 void PrerenderHistograms::RecordConcurrency(size_t prerender_count, 170 void PrerenderHistograms::RecordConcurrency(size_t prerender_count) const {
171 size_t max_concurrency) const { 171 static const size_t kMaxRecordableConcurrency = 20;
172 static const size_t kMaxRecordableConcurrency = 3; 172 DCHECK_GE(kMaxRecordableConcurrency, Config().max_link_concurrency);
173 DCHECK_GE(kMaxRecordableConcurrency, max_concurrency); 173 UMA_HISTOGRAM_ENUMERATION(
174 if (max_concurrency > 1) { 174 StringPrintf("Prerender.PrerenderCountOf%" PRIuS "Max",
175 UMA_HISTOGRAM_ENUMERATION( 175 kMaxRecordableConcurrency),
176 StringPrintf("Prerender.PrerenderCountOf%" PRIuS "Max", 176 prerender_count, kMaxRecordableConcurrency + 1);
177 max_concurrency),
178 prerender_count, kMaxRecordableConcurrency + 1);
179 }
180 } 177 }
181 178
182 void PrerenderHistograms::RecordUsedPrerender(Origin origin) const { 179 void PrerenderHistograms::RecordUsedPrerender(Origin origin) const {
183 if (OriginIsOmnibox(origin)) { 180 if (OriginIsOmnibox(origin)) {
184 UMA_HISTOGRAM_ENUMERATION( 181 UMA_HISTOGRAM_ENUMERATION(
185 StringPrintf("Prerender.OmniboxNavigationsUsedPrerenderCount%s", 182 StringPrintf("Prerender.OmniboxNavigationsUsedPrerenderCount%s",
186 PrerenderManager::GetModeString()), 1, 2); 183 PrerenderManager::GetModeString()), 1, 2);
187 } 184 }
188 } 185 }
189 186
(...skipping 200 matching lines...) Expand 10 before | Expand all | Expand 10 after
390 return last_experiment_id_; 387 return last_experiment_id_;
391 } 388 }
392 389
393 bool PrerenderHistograms::IsOriginExperimentWash() const { 390 bool PrerenderHistograms::IsOriginExperimentWash() const {
394 if (!WithinWindow()) 391 if (!WithinWindow())
395 return false; 392 return false;
396 return origin_experiment_wash_; 393 return origin_experiment_wash_;
397 } 394 }
398 395
399 } // namespace prerender 396 } // namespace prerender
OLDNEW
« 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