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

Side by Side Diff: chrome/browser/prerender/prerender_histograms.cc

Issue 10453013: Remove hit weight experiment (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: trying rebase again - missing base file? Created 8 years, 6 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/metrics/field_trial.h" 9 #include "base/metrics/field_trial.h"
10 #include "base/metrics/histogram.h" 10 #include "base/metrics/histogram.h"
(...skipping 30 matching lines...) Expand all
41 return ComposeHistogramName("gws", name); 41 return ComposeHistogramName("gws", name);
42 return ComposeHistogramName("exp" + std::string(1, experiment_id + '0'), 42 return ComposeHistogramName("exp" + std::string(1, experiment_id + '0'),
43 name); 43 name);
44 } 44 }
45 45
46 if (experiment_id != kNoExperiment) 46 if (experiment_id != kNoExperiment)
47 return ComposeHistogramName("wash", name); 47 return ComposeHistogramName("wash", name);
48 48
49 switch (origin) { 49 switch (origin) {
50 case ORIGIN_OMNIBOX: 50 case ORIGIN_OMNIBOX:
51 return ComposeHistogramName( 51 return ComposeHistogramName("omnibox", name);
52 StringPrintf("omnibox_%.1f",
53 AutocompleteActionPredictor::get_hit_weight()).c_str(),
54 name);
55 case ORIGIN_LINK_REL_PRERENDER: 52 case ORIGIN_LINK_REL_PRERENDER:
56 return ComposeHistogramName("web", name); 53 return ComposeHistogramName("web", name);
57 case ORIGIN_GWS_PRERENDER: // Handled above. 54 case ORIGIN_GWS_PRERENDER: // Handled above.
58 default: 55 default:
59 NOTREACHED(); 56 NOTREACHED();
60 break; 57 break;
61 }; 58 };
62 59
63 // Dummy return value to make the compiler happy. 60 // Dummy return value to make the compiler happy.
64 NOTREACHED(); 61 NOTREACHED();
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
147 // reset the window to begin at the most recent occurrence, so that we will 144 // reset the window to begin at the most recent occurrence, so that we will
148 // always be in a window in the 30 seconds from each occurrence. 145 // always be in a window in the 30 seconds from each occurrence.
149 last_prerender_seen_time_ = GetCurrentTimeTicks(); 146 last_prerender_seen_time_ = GetCurrentTimeTicks();
150 seen_any_pageload_ = false; 147 seen_any_pageload_ = false;
151 seen_pageload_started_after_prerender_ = false; 148 seen_pageload_started_after_prerender_ = false;
152 } 149 }
153 150
154 void PrerenderHistograms::RecordPrerenderStarted(Origin origin) const { 151 void PrerenderHistograms::RecordPrerenderStarted(Origin origin) const {
155 if (OriginIsOmnibox(origin)) { 152 if (OriginIsOmnibox(origin)) {
156 UMA_HISTOGRAM_COUNTS( 153 UMA_HISTOGRAM_COUNTS(
157 StringPrintf("Prerender.OmniboxPrerenderCount_%.1f%s", 154 StringPrintf("Prerender.OmniboxPrerenderCount_%s",
158 AutocompleteActionPredictor::get_hit_weight(),
159 PrerenderManager::GetModeString()).c_str(), 155 PrerenderManager::GetModeString()).c_str(),
160 1); 156 1);
161 } 157 }
162 } 158 }
163 159
164 void PrerenderHistograms::RecordUsedPrerender(Origin origin) const { 160 void PrerenderHistograms::RecordUsedPrerender(Origin origin) const {
165 if (OriginIsOmnibox(origin)) { 161 if (OriginIsOmnibox(origin)) {
166 UMA_HISTOGRAM_COUNTS( 162 UMA_HISTOGRAM_COUNTS(
167 StringPrintf("Prerender.OmniboxNavigationsUsedPrerenderCount_%.1f%s", 163 StringPrintf("Prerender.OmniboxNavigationsUsedPrerenderCount_%s",
168 AutocompleteActionPredictor::get_hit_weight(),
169 PrerenderManager::GetModeString()).c_str(), 164 PrerenderManager::GetModeString()).c_str(),
170 1); 165 1);
171 } 166 }
172 } 167 }
173 168
174 void PrerenderHistograms::RecordTimeSinceLastRecentVisit( 169 void PrerenderHistograms::RecordTimeSinceLastRecentVisit(
175 base::TimeDelta delta) const { 170 base::TimeDelta delta) const {
176 PREFIXED_HISTOGRAM( 171 PREFIXED_HISTOGRAM(
177 "TimeSinceLastRecentVisit", 172 "TimeSinceLastRecentVisit",
178 UMA_HISTOGRAM_TIMES(name, delta)); 173 UMA_HISTOGRAM_TIMES(name, delta));
(...skipping 202 matching lines...) Expand 10 before | Expand all | Expand 10 after
381 return last_origin_; 376 return last_origin_;
382 } 377 }
383 378
384 bool PrerenderHistograms::IsOriginExperimentWash() const { 379 bool PrerenderHistograms::IsOriginExperimentWash() const {
385 if (!WithinWindow()) 380 if (!WithinWindow())
386 return false; 381 return false;
387 return origin_experiment_wash_; 382 return origin_experiment_wash_;
388 } 383 }
389 384
390 } // namespace prerender 385 } // namespace prerender
OLDNEW
« no previous file with comments | « chrome/browser/prerender/prerender_field_trial.cc ('k') | chrome/browser/resources/predictors/predictors.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698