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

Unified Diff: chrome/browser/background/background_application_list_model.cc

Issue 10649003: Move each permission classes to its own files in extensions/permissions (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Rebase on HEAD Created 8 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/background/background_application_list_model.cc
diff --git a/chrome/browser/background/background_application_list_model.cc b/chrome/browser/background/background_application_list_model.cc
index 3501404c9fcb66606eced58ad4d456e78dec8e5e..4e93935309da592e39fe1d34fa78c516a0043a06 100644
--- a/chrome/browser/background/background_application_list_model.cc
+++ b/chrome/browser/background/background_application_list_model.cc
@@ -28,8 +28,10 @@
#include "ui/gfx/image/image.h"
#include "ui/gfx/image/image_skia.h"
+using extensions::APIPermission;
using extensions::Extension;
using extensions::ExtensionList;
+using extensions::PermissionSet;
using extensions::UnloadedExtensionInfo;
using extensions::UpdatedExtensionPermissionsInfo;
@@ -274,7 +276,7 @@ bool BackgroundApplicationListModel::IsBackgroundApp(
// manifest.
// Not a background app if we don't have the background permission.
- if (!extension.HasAPIPermission(ExtensionAPIPermission::kBackground))
+ if (!extension.HasAPIPermission(APIPermission::kBackground))
return false;
// Extensions and packaged apps with background permission are always treated
@@ -363,8 +365,8 @@ void BackgroundApplicationListModel::OnExtensionUnloaded(
void BackgroundApplicationListModel::OnExtensionPermissionsUpdated(
const Extension* extension,
UpdatedExtensionPermissionsInfo::Reason reason,
- const ExtensionPermissionSet* permissions) {
- if (permissions->HasAPIPermission(ExtensionAPIPermission::kBackground)) {
+ const PermissionSet* permissions) {
+ if (permissions->HasAPIPermission(APIPermission::kBackground)) {
switch (reason) {
case UpdatedExtensionPermissionsInfo::ADDED:
DCHECK(IsBackgroundApp(*extension, profile_));

Powered by Google App Engine
This is Rietveld 408576698