Chromium Code Reviews| Index: chrome/browser/permissions/permission_uma_util.cc |
| diff --git a/chrome/browser/permissions/permission_uma_util.cc b/chrome/browser/permissions/permission_uma_util.cc |
| index ee69985714e9488e9ed9e9af6eb372a8e6dec8ae..8ae9ef0b4b96cf3f021b43423e6b519216f38227 100644 |
| --- a/chrome/browser/permissions/permission_uma_util.cc |
| +++ b/chrome/browser/permissions/permission_uma_util.cc |
| @@ -356,6 +356,47 @@ void PermissionUmaUtil::PermissionPromptDenied( |
| requests[0]->GetGestureType(), requests[0]->GetPermissionRequestType()); |
| } |
| +void PermissionUmaUtil::PermissionPromptDismissCount( |
| + content::PermissionType permission, |
| + int count) { |
| + switch (permission) { |
| + case PermissionType::GEOLOCATION: |
| + UMA_HISTOGRAM_COUNTS_100("Permissions.DismissCount.Geolocation", count); |
| + break; |
| + case PermissionType::NOTIFICATIONS: |
| + UMA_HISTOGRAM_COUNTS_100("Permissions.DismissCount.Notifications", count); |
| + break; |
| + case PermissionType::MIDI_SYSEX: |
| + UMA_HISTOGRAM_COUNTS_100("Permissions.DismissCount.MidiSysEx", count); |
| + break; |
| + case PermissionType::PUSH_MESSAGING: |
| + UMA_HISTOGRAM_COUNTS_100("Permissions.DismissCount.PushMessaging", count); |
| + break; |
| + case PermissionType::PROTECTED_MEDIA_IDENTIFIER: |
| + UMA_HISTOGRAM_COUNTS_100("Permissions.DismissCount.ProtectedMedia", |
| + count); |
| + break; |
| + case PermissionType::DURABLE_STORAGE: |
| + UMA_HISTOGRAM_COUNTS_100("Permissions.DismissCount.DurableStorage", |
| + count); |
| + break; |
| + case PermissionType::AUDIO_CAPTURE: |
| + UMA_HISTOGRAM_COUNTS_100("Permissions.DismissCount.AudioCapture", count); |
| + break; |
| + case PermissionType::VIDEO_CAPTURE: |
| + UMA_HISTOGRAM_COUNTS_100("Permissions.DismissCount.VideoCapture", count); |
| + break; |
| + // The user is not prompted for these permissions, thus there is no |
| + // permission action recorded for them. |
|
raymes
2016/08/05 03:27:48
nit: this comment is inaccurate (no dismiss record
dominickn
2016/08/05 04:29:30
Done.
|
| + case PermissionType::MIDI: |
| + case PermissionType::BACKGROUND_SYNC: |
| + case PermissionType::NUM: |
| + NOTREACHED() << "PERMISSION " |
| + << PermissionUtil::GetPermissionString(permission) |
| + << " not accounted for"; |
| + } |
| +} |
| + |
| bool PermissionUmaUtil::IsOptedIntoPermissionActionReporting(Profile* profile) { |
| if (!base::CommandLine::ForCurrentProcess()->HasSwitch( |
| switches::kEnablePermissionActionReporting)) |