Chromium Code Reviews| Index: chrome/browser/permissions/permission_queue_controller.cc |
| diff --git a/chrome/browser/permissions/permission_queue_controller.cc b/chrome/browser/permissions/permission_queue_controller.cc |
| index 4a126626edb7195011b91c3f910c805f7e4c6884..6f12eb470f49a761e9ae0eda98b5ab19b7407054 100644 |
| --- a/chrome/browser/permissions/permission_queue_controller.cc |
| +++ b/chrome/browser/permissions/permission_queue_controller.cc |
| @@ -8,6 +8,7 @@ |
| #include "chrome/browser/chrome_notification_types.h" |
| #include "chrome/browser/content_settings/host_content_settings_map_factory.h" |
| #include "chrome/browser/infobars/infobar_service.h" |
| +#include "chrome/browser/permissions/permission_decision_auto_blocker.h" |
| #include "chrome/browser/permissions/permission_dialog_delegate.h" |
| #include "chrome/browser/permissions/permission_infobar_delegate.h" |
| #include "chrome/browser/permissions/permission_request.h" |
| @@ -228,16 +229,28 @@ void PermissionQueueController::OnPermissionSet(const PermissionRequestID& id, |
| requesting_frame, profile_); |
| PermissionUmaUtil::RecordPermissionPromptAccepted(request_type, |
| gesture_type); |
| + PermissionUmaUtil::RecordPermissionEmbargoStatus( |
| + PermissionEmbargoStatus::NOT_EMBARGOED); |
| break; |
| case DENIED: |
| PermissionUmaUtil::PermissionDenied(permission_type_, gesture_type, |
| requesting_frame, profile_); |
| PermissionUmaUtil::RecordPermissionPromptDenied(request_type, |
| gesture_type); |
| + PermissionUmaUtil::RecordPermissionEmbargoStatus( |
| + PermissionEmbargoStatus::NOT_EMBARGOED); |
| break; |
| case DISMISSED: |
| PermissionUmaUtil::PermissionDismissed(permission_type_, gesture_type, |
| requesting_frame, profile_); |
| + if (PermissionDecisionAutoBlocker::GetForProfile(profile_) |
| + ->RecordDismissAndEmbargo(requesting_frame, permission_type_)) { |
|
raymes
2017/02/13 05:00:58
Adding some Java tests sounds good to me :) Can we
dominickn
2017/02/13 05:24:47
Filed crbug.com/691437
|
| + PermissionUmaUtil::RecordPermissionEmbargoStatus( |
| + PermissionEmbargoStatus::REPEATED_DISMISSALS); |
| + } else { |
| + PermissionUmaUtil::RecordPermissionEmbargoStatus( |
| + PermissionEmbargoStatus::NOT_EMBARGOED); |
| + } |
| break; |
| default: |
| NOTREACHED(); |