| 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 2b7ff8f18f6d8c071bba34f6ffa245493b4fe91e..2984eebef11ddf1b7550f64b8006f774fe37a3e1 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,9 +38,11 @@ void RecordPermissionAction(
|
| ContentSettingsType permission, PermissionAction action) {
|
| switch (permission) {
|
| case CONTENT_SETTINGS_TYPE_GEOLOCATION:
|
| - // TODO(miguelg): support geolocation through
|
| - // the generic permission class.
|
| - break;
|
| + UMA_HISTOGRAM_ENUMERATION(
|
| + "ContentSettings.PermisionActions_Geolocation",
|
| + action,
|
| + PERMISSION_ACTION_NUM);
|
| +
|
| case CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
|
| UMA_HISTOGRAM_ENUMERATION(
|
| "ContentSettings.PermisionActions_Notifications",
|
| @@ -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;
|
|
|