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

Side by Side Diff: chrome/browser/permissions/permission_uma_util.h

Issue 2250993002: Add prior dismissal and ignore count metrics for all permission actions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@kendra-permission-action-reporting
Patch Set: Comments Created 4 years, 4 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_ 5 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_
6 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_ 6 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 static const char kPermissionsPromptAccepted[]; 73 static const char kPermissionsPromptAccepted[];
74 static const char kPermissionsPromptAcceptedGesture[]; 74 static const char kPermissionsPromptAcceptedGesture[];
75 static const char kPermissionsPromptAcceptedNoGesture[]; 75 static const char kPermissionsPromptAcceptedNoGesture[];
76 static const char kPermissionsPromptDenied[]; 76 static const char kPermissionsPromptDenied[];
77 static const char kPermissionsPromptDeniedGesture[]; 77 static const char kPermissionsPromptDeniedGesture[];
78 static const char kPermissionsPromptDeniedNoGesture[]; 78 static const char kPermissionsPromptDeniedNoGesture[];
79 static const char kPermissionsPromptRequestsPerPrompt[]; 79 static const char kPermissionsPromptRequestsPerPrompt[];
80 static const char kPermissionsPromptMergedBubbleTypes[]; 80 static const char kPermissionsPromptMergedBubbleTypes[];
81 static const char kPermissionsPromptMergedBubbleAccepted[]; 81 static const char kPermissionsPromptMergedBubbleAccepted[];
82 static const char kPermissionsPromptMergedBubbleDenied[]; 82 static const char kPermissionsPromptMergedBubbleDenied[];
83 static const char kPermissionsPromptAcceptedPriorDismissCountPrefix[];
84 static const char kPermissionsPromptAcceptedPriorIgnoreCountPrefix[];
85 static const char kPermissionsPromptDeniedPriorDismissCountPrefix[];
86 static const char kPermissionsPromptDeniedPriorIgnoreCountPrefix[];
87 static const char kPermissionsPromptDismissedPriorDismissCountPrefix[];
88 static const char kPermissionsPromptDismissedPriorIgnoreCountPrefix[];
89 static const char kPermissionsPromptIgnoredPriorDismissCountPrefix[];
90 static const char kPermissionsPromptIgnoredPriorIgnoreCountPrefix[];
83 91
84 static void PermissionRequested(content::PermissionType permission, 92 static void PermissionRequested(content::PermissionType permission,
85 const GURL& requesting_origin, 93 const GURL& requesting_origin,
86 const GURL& embedding_origin, 94 const GURL& embedding_origin,
87 Profile* profile); 95 Profile* profile);
88 static void PermissionGranted(content::PermissionType permission, 96 static void PermissionGranted(content::PermissionType permission,
89 PermissionRequestGestureType gesture_type, 97 PermissionRequestGestureType gesture_type,
90 const GURL& requesting_origin, 98 const GURL& requesting_origin,
91 Profile* profile); 99 Profile* profile);
92 static void PermissionDenied(content::PermissionType permission, 100 static void PermissionDenied(content::PermissionType permission,
(...skipping 30 matching lines...) Expand all
123 // Note that for coalesced permission bubbles, PermissionPromptAccepted will 131 // Note that for coalesced permission bubbles, PermissionPromptAccepted will
124 // always be called, with |accept_states| containing whether each request was 132 // always be called, with |accept_states| containing whether each request was
125 // accepted or denied. 133 // accepted or denied.
126 static void PermissionPromptAccepted( 134 static void PermissionPromptAccepted(
127 const std::vector<PermissionRequest*>& requests, 135 const std::vector<PermissionRequest*>& requests,
128 const std::vector<bool>& accept_states); 136 const std::vector<bool>& accept_states);
129 137
130 static void PermissionPromptDenied( 138 static void PermissionPromptDenied(
131 const std::vector<PermissionRequest*>& requests); 139 const std::vector<PermissionRequest*>& requests);
132 140
133 // Records |count| total dismissal actions for a prompt of type |permission|
134 // for a single origin.
135 static void PermissionPromptDismissed(content::PermissionType permission,
136 int count);
137
138 // Records |count| total ignore actions for a prompt of type |permission| for
139 // a single origin.
140 static void PermissionPromptIgnored(content::PermissionType permission,
141 int count);
142
143 // A permission prompt was accepted or denied, and the prompt displayed a 141 // A permission prompt was accepted or denied, and the prompt displayed a
144 // persistence toggle. Records whether the toggle was enabled (persist) or 142 // persistence toggle. Records whether the toggle was enabled (persist) or
145 // disabled (don't persist). 143 // disabled (don't persist).
146 static void PermissionPromptAcceptedWithPersistenceToggle( 144 static void PermissionPromptAcceptedWithPersistenceToggle(
147 content::PermissionType permission, 145 content::PermissionType permission,
148 bool toggle_enabled); 146 bool toggle_enabled);
149 147
150 static void PermissionPromptDeniedWithPersistenceToggle( 148 static void PermissionPromptDeniedWithPersistenceToggle(
151 content::PermissionType permission, 149 content::PermissionType permission,
152 bool toggle_enabled); 150 bool toggle_enabled);
153 151
154 private: 152 private:
155 friend class PermissionUmaUtilTest; 153 friend class PermissionUmaUtilTest;
156 154
157 static bool IsOptedIntoPermissionActionReporting(Profile* profile); 155 static bool IsOptedIntoPermissionActionReporting(Profile* profile);
158 156
159 static void RecordPermissionAction(content::PermissionType permission, 157 static void RecordPermissionAction(content::PermissionType permission,
160 PermissionAction action, 158 PermissionAction action,
161 PermissionSourceUI source_ui, 159 PermissionSourceUI source_ui,
162 PermissionRequestGestureType gesture_type, 160 PermissionRequestGestureType gesture_type,
163 const GURL& requesting_origin, 161 const GURL& requesting_origin,
164 Profile* profile); 162 Profile* profile);
165 163
164 // Records |count| total prior actions for a prompt of type |permission|
165 // for a single origin using |prefix| for the metric.
166 static void RecordPermissionPromptPriorCount(
167 content::PermissionType permission,
168 const std::string& prefix,
169 int count);
170
166 DISALLOW_IMPLICIT_CONSTRUCTORS(PermissionUmaUtil); 171 DISALLOW_IMPLICIT_CONSTRUCTORS(PermissionUmaUtil);
167 }; 172 };
168 173
169 #endif // CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_ 174 #endif // CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698