| Index: chrome/browser/content_settings/permission_context_uma_util.cc
|
| diff --git a/chrome/browser/content_settings/permission_context_uma_util.cc b/chrome/browser/content_settings/permission_context_uma_util.cc
|
| index 0b16be020ee05b746ed15dc06bc2e5fb998b8a63..b76d5414a384ea1f6f85f4c9749ab80a96c73341 100644
|
| --- a/chrome/browser/content_settings/permission_context_uma_util.cc
|
| +++ b/chrome/browser/content_settings/permission_context_uma_util.cc
|
| @@ -28,6 +28,7 @@ enum PermissionType {
|
| PERMISSION_MIDI_SYSEX = 1,
|
| PERMISSION_PUSH_MESSAGING = 2,
|
| PERMISSION_NOTIFICATIONS = 3,
|
| + PERMISSION_GEOLOCATION = 4,
|
|
|
| // Always keep this at the end.
|
| PERMISSION_NUM,
|
| @@ -37,8 +38,10 @@ void RecordPermissionAction(
|
| ContentSettingsType permission, PermissionAction action) {
|
| switch (permission) {
|
| case CONTENT_SETTINGS_TYPE_GEOLOCATION:
|
| - // TODO(miguelg): support geolocation through
|
| - // the generic permission class.
|
| + UMA_HISTOGRAM_ENUMERATION(
|
| + "ContentSettings.PermissionActions_Geolocation",
|
| + action,
|
| + PERMISSION_ACTION_NUM);
|
| break;
|
| case CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
|
| UMA_HISTOGRAM_ENUMERATION(
|
| @@ -72,6 +75,9 @@ void RecordPermissionRequest(
|
| ContentSettingsType permission) {
|
| PermissionType type;
|
| switch (permission) {
|
| + case CONTENT_SETTINGS_TYPE_GEOLOCATION:
|
| + type = PERMISSION_GEOLOCATION;
|
| + break;
|
| case CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
|
| type = PERMISSION_NOTIFICATIONS;
|
| break;
|
|
|