Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(947)

Unified Diff: chrome/browser/permissions/permission_uma_util.cc

Issue 2936003003: MD Settings: Set all content setting values in Site Details Javascript. (Closed)
Patch Set: Review comments, tests & cleanup. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 45305b15c1a75ddd61e955a735d4e95ff0861ab9..75479c672e514c462914793e39ff12c6c41d0128 100644
--- a/chrome/browser/permissions/permission_uma_util.cc
+++ b/chrome/browser/permissions/permission_uma_util.cc
@@ -340,8 +340,11 @@ void PermissionUmaUtil::RecordEmbargoPromptSuppressionFromSource(
PermissionUmaUtil::RecordEmbargoPromptSuppression(
PermissionEmbargoStatus::PERMISSIONS_BLACKLISTING);
break;
- case PermissionStatusSource::UNSPECIFIED:
case PermissionStatusSource::KILL_SWITCH:
+ case PermissionStatusSource::INSECURE_ORIGIN:
+ case PermissionStatusSource::ENTERPRISE_POLICY:
+ case PermissionStatusSource::EXTENSION:
+ case PermissionStatusSource::UNSPECIFIED:
// The permission wasn't under embargo, so don't record anything. We may
// embargo it later.
break;

Powered by Google App Engine
This is Rietveld 408576698