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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/safe_browsing/permission_reporter.cc

Issue 2339093002: [HBD] Add Plugins permission type and PermissionContext (Closed)
Patch Set: Created 4 years, 3 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
« chrome/browser/permissions/permission_manager.cc ('K') | « chrome/browser/plugins/plugins_permission_context.cc ('k') | chrome/common/safe_browsing/permission_report.proto » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/safe_browsing/permission_reporter.cc
diff --git a/chrome/browser/safe_browsing/permission_reporter.cc b/chrome/browser/safe_browsing/permission_reporter.cc
index 2a9c0e449f208ecb964daaa1cc2b18d75bd51f07..855270e9b2a989b3dc2e72714755ded28b125036 100644
--- a/chrome/browser/safe_browsing/permission_reporter.cc
+++ b/chrome/browser/safe_browsing/permission_reporter.cc
@@ -49,7 +49,9 @@ PermissionReport::PermissionType PermissionTypeForReport(
case PermissionType::VIDEO_CAPTURE:
return PermissionReport::VIDEO_CAPTURE;
case PermissionType::BACKGROUND_SYNC:
- return PermissionReport::UNKNOWN_PERMISSION;
+ return PermissionReport::BACKGROUND_SYNC;
+ case PermissionType::PLUGINS:
+ return PermissionReport::PLUGINS;
case PermissionType::NUM:
break;
}
« chrome/browser/permissions/permission_manager.cc ('K') | « chrome/browser/plugins/plugins_permission_context.cc ('k') | chrome/common/safe_browsing/permission_report.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698