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

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

Issue 2715643002: Replace all enums with enum classes in permissions code. (Closed)
Patch Set: Rebase Created 3 years, 10 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/safe_browsing/permission_reporter.cc
diff --git a/chrome/browser/safe_browsing/permission_reporter.cc b/chrome/browser/safe_browsing/permission_reporter.cc
index 6af846e61e00e181924f3d9ef148470d35169a3e..ba00b2fdd80a61e68b07177db25b2439920c962d 100644
--- a/chrome/browser/safe_browsing/permission_reporter.cc
+++ b/chrome/browser/safe_browsing/permission_reporter.cc
@@ -57,20 +57,20 @@ PermissionReport::PermissionType PermissionTypeForReport(
PermissionReport::Action PermissionActionForReport(PermissionAction action) {
switch (action) {
- case GRANTED:
+ case PermissionAction::GRANTED:
return PermissionReport::GRANTED;
- case DENIED:
+ case PermissionAction::DENIED:
return PermissionReport::DENIED;
- case DISMISSED:
+ case PermissionAction::DISMISSED:
return PermissionReport::DISMISSED;
- case IGNORED:
+ case PermissionAction::IGNORED:
return PermissionReport::IGNORED;
- case REVOKED:
+ case PermissionAction::REVOKED:
return PermissionReport::REVOKED;
- case REENABLED:
- case REQUESTED:
+ case PermissionAction::REENABLED:
+ case PermissionAction::REQUESTED:
return PermissionReport::ACTION_UNSPECIFIED;
- case PERMISSION_ACTION_NUM:
+ case PermissionAction::NUM:
break;
}
@@ -88,7 +88,7 @@ PermissionReport::SourceUI SourceUIForReport(PermissionSourceUI source_ui) {
return PermissionReport::SITE_SETTINGS;
case PermissionSourceUI::PAGE_ACTION:
return PermissionReport::PAGE_ACTION;
- case PermissionSourceUI::SOURCE_UI_NUM:
+ case PermissionSourceUI::NUM:
break;
}
« no previous file with comments | « chrome/browser/permissions/permission_util_unittest.cc ('k') | chrome/browser/safe_browsing/permission_reporter_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698