| 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 b467bac974f1dd8deb951717af720e200c7a4eeb..ef3b305f042abddc8b9f6f92616f8d7d1ae4ed1c 100644
|
| --- a/chrome/browser/permissions/permission_uma_util.cc
|
| +++ b/chrome/browser/permissions/permission_uma_util.cc
|
| @@ -196,30 +196,39 @@ void PermissionUmaUtil::PermissionRequested(PermissionType permission,
|
|
|
| void PermissionUmaUtil::PermissionGranted(PermissionType permission,
|
| const GURL& requesting_origin,
|
| + SourceUI source_ui,
|
| Profile* profile) {
|
| - RecordPermissionAction(permission, GRANTED, requesting_origin, profile);
|
| + RecordPermissionAction(permission, GRANTED, requesting_origin, source_ui,
|
| + profile);
|
| }
|
|
|
| void PermissionUmaUtil::PermissionDenied(PermissionType permission,
|
| const GURL& requesting_origin,
|
| + SourceUI source_ui,
|
| Profile* profile) {
|
| - RecordPermissionAction(permission, DENIED, requesting_origin, profile);
|
| + RecordPermissionAction(permission, DENIED, requesting_origin, source_ui,
|
| + profile);
|
| }
|
|
|
| void PermissionUmaUtil::PermissionDismissed(PermissionType permission,
|
| const GURL& requesting_origin,
|
| + SourceUI source_ui,
|
| Profile* profile) {
|
| - RecordPermissionAction(permission, DISMISSED, requesting_origin, profile);
|
| + RecordPermissionAction(permission, DISMISSED, requesting_origin, source_ui,
|
| + profile);
|
| }
|
|
|
| void PermissionUmaUtil::PermissionIgnored(PermissionType permission,
|
| const GURL& requesting_origin,
|
| + SourceUI source_ui,
|
| Profile* profile) {
|
| - RecordPermissionAction(permission, IGNORED, requesting_origin, profile);
|
| + RecordPermissionAction(permission, IGNORED, requesting_origin, source_ui,
|
| + profile);
|
| }
|
|
|
| void PermissionUmaUtil::PermissionRevoked(PermissionType permission,
|
| const GURL& revoked_origin,
|
| + SourceUI source_ui,
|
| Profile* profile) {
|
| // TODO(tsergeant): Expand metrics definitions for revocation to include all
|
| // permissions.
|
| @@ -227,7 +236,8 @@ void PermissionUmaUtil::PermissionRevoked(PermissionType permission,
|
| permission == PermissionType::GEOLOCATION ||
|
| permission == PermissionType::AUDIO_CAPTURE ||
|
| permission == PermissionType::VIDEO_CAPTURE) {
|
| - RecordPermissionAction(permission, REVOKED, revoked_origin, profile);
|
| + RecordPermissionAction(permission, REVOKED, revoked_origin, source_ui,
|
| + profile);
|
| }
|
| }
|
|
|
| @@ -332,11 +342,13 @@ bool PermissionUmaUtil::IsOptedIntoPermissionActionReporting(Profile* profile) {
|
| void PermissionUmaUtil::RecordPermissionAction(PermissionType permission,
|
| PermissionAction action,
|
| const GURL& requesting_origin,
|
| + SourceUI source_ui,
|
| Profile* profile) {
|
| if (IsOptedIntoPermissionActionReporting(profile)) {
|
| g_browser_process->safe_browsing_service()
|
| ->ui_manager()
|
| - ->ReportPermissionAction(requesting_origin, permission, action);
|
| + ->ReportPermissionAction(requesting_origin, permission, action,
|
| + source_ui);
|
| }
|
|
|
| bool secure_origin = content::IsOriginSecure(requesting_origin);
|
|
|