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

Side by Side Diff: chrome/browser/banners/app_banner_metrics.cc

Issue 2122783002: Add metrics for app banner preventDefault() and prompt(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use NO_ACTION Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/banners/app_banner_metrics.h" 5 #include "chrome/browser/banners/app_banner_metrics.h"
6 6
7 #include "base/metrics/histogram_macros.h" 7 #include "base/metrics/histogram_macros.h"
8 #include "base/metrics/sparse_histogram.h" 8 #include "base/metrics/sparse_histogram.h"
9 9
10 namespace banners { 10 namespace banners {
11 11
12 const char kDismissEventHistogram[] = "AppBanners.DismissEvent"; 12 const char kDismissEventHistogram[] = "AppBanners.DismissEvent";
13 const char kDisplayEventHistogram[] = "AppBanners.DisplayEvent"; 13 const char kDisplayEventHistogram[] = "AppBanners.DisplayEvent";
14 const char kInstallEventHistogram[] = "AppBanners.InstallEvent"; 14 const char kInstallEventHistogram[] = "AppBanners.InstallEvent";
15 const char kMinutesHistogram[] = 15 const char kMinutesHistogram[] =
16 "AppBanners.MinutesFromFirstVisitToBannerShown"; 16 "AppBanners.MinutesFromFirstVisitToBannerShown";
17 const char kUserResponseHistogram[] = "AppBanners.UserResponse"; 17 const char kUserResponseHistogram[] = "AppBanners.UserResponse";
18 const char kBeforeInstallEventHistogram[] = "AppBanners.BeforeInstallEvent";
18 19
19 void TrackDismissEvent(int event) { 20 void TrackDismissEvent(int event) {
20 DCHECK_LT(DISMISS_EVENT_MIN, event); 21 DCHECK_LT(DISMISS_EVENT_MIN, event);
21 DCHECK_LT(event, DISMISS_EVENT_MAX); 22 DCHECK_LT(event, DISMISS_EVENT_MAX);
22 UMA_HISTOGRAM_SPARSE_SLOWLY(kDismissEventHistogram, event); 23 UMA_HISTOGRAM_SPARSE_SLOWLY(kDismissEventHistogram, event);
23 } 24 }
24 25
25 void TrackDisplayEvent(int event) { 26 void TrackDisplayEvent(int event) {
26 DCHECK_LT(DISPLAY_EVENT_MIN, event); 27 DCHECK_LT(DISPLAY_EVENT_MIN, event);
27 DCHECK_LT(event, DISPLAY_EVENT_MAX); 28 DCHECK_LT(event, DISPLAY_EVENT_MAX);
(...skipping 13 matching lines...) Expand all
41 // navigation heuristic. 42 // navigation heuristic.
42 UMA_HISTOGRAM_CUSTOM_COUNTS(kMinutesHistogram, minutes, 1, 30240, 100); 43 UMA_HISTOGRAM_CUSTOM_COUNTS(kMinutesHistogram, minutes, 1, 30240, 100);
43 } 44 }
44 45
45 void TrackUserResponse(int event) { 46 void TrackUserResponse(int event) {
46 DCHECK_LT(USER_RESPONSE_MIN, event); 47 DCHECK_LT(USER_RESPONSE_MIN, event);
47 DCHECK_LT(event, USER_RESPONSE_MAX); 48 DCHECK_LT(event, USER_RESPONSE_MAX);
48 UMA_HISTOGRAM_SPARSE_SLOWLY(kUserResponseHistogram, event); 49 UMA_HISTOGRAM_SPARSE_SLOWLY(kUserResponseHistogram, event);
49 } 50 }
50 51
52 void TrackBeforeInstallEvent(int event) {
53 DCHECK_LT(BEFORE_INSTALL_EVENT_MIN, event);
54 DCHECK_LT(event, BEFORE_INSTALL_EVENT_MAX);
55 UMA_HISTOGRAM_SPARSE_SLOWLY(kBeforeInstallEventHistogram, event);
56 }
57
51 } // namespace banners 58 } // namespace banners
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698