summaryrefslogtreecommitdiffstats
path: root/extensions/common/permissions
diff options
context:
space:
mode:
authortreib <treib@chromium.org>2015-08-27 09:32:24 -0700
committerCommit bot <commit-bot@chromium.org>2015-08-27 16:33:16 +0000
commitf9bde4c57f6070dde911bb060e114e6da770f4d3 (patch)
treee5e210d9ddf261c0e8fd468582583fc96a2f9ac3 /extensions/common/permissions
parent74165fab66c8804876a6617ce0c74bd37679c259 (diff)
downloadchromium_src-f9bde4c57f6070dde911bb060e114e6da770f4d3.zip
chromium_src-f9bde4c57f6070dde911bb060e114e6da770f4d3.tar.gz
chromium_src-f9bde4c57f6070dde911bb060e114e6da770f4d3.tar.bz2
Remove APIPermission::kFileSystemWriteDirectory
In the new permission system, the corresponding message can be represented by kFileSystemWrite + kFileSystemDirectory, we don't need a special fake ID anymore. Skipping presubmit because of renamed histogram entry. NOPRESUBMIT=true BUG=284849 Review URL: https://codereview.chromium.org/1217773006 Cr-Commit-Position: refs/heads/master@{#345897}
Diffstat (limited to 'extensions/common/permissions')
-rw-r--r--extensions/common/permissions/api_permission.h2
-rw-r--r--extensions/common/permissions/api_permission_set.cc11
-rw-r--r--extensions/common/permissions/api_permission_set.h2
3 files changed, 1 insertions, 14 deletions
diff --git a/extensions/common/permissions/api_permission.h b/extensions/common/permissions/api_permission.h
index b2c6d46..e49a153 100644
--- a/extensions/common/permissions/api_permission.h
+++ b/extensions/common/permissions/api_permission.h
@@ -118,7 +118,7 @@ class APIPermission {
kFileSystemRequestFileSystem,
kFileSystemRetainEntries,
kFileSystemWrite,
- kFileSystemWriteDirectory,
+ kDeleted_FileSystemWriteDirectory,
kFirstRunPrivate,
kFontSettings,
kFullscreen,
diff --git a/extensions/common/permissions/api_permission_set.cc b/extensions/common/permissions/api_permission_set.cc
index 74a3e66..9f6df2f 100644
--- a/extensions/common/permissions/api_permission_set.cc
+++ b/extensions/common/permissions/api_permission_set.cc
@@ -182,17 +182,6 @@ bool APIPermissionSet::ParseFromJSON(
return true;
}
-void APIPermissionSet::AddImpliedPermissions() {
- // The fileSystem.write and fileSystem.directory permissions imply
- // fileSystem.writeDirectory.
- // Has a corresponding rule in ChromePermissionMessageProvider.
- // TODO(sammc): Remove this. See http://crbug.com/284849.
- if (ContainsKey(map(), APIPermission::kFileSystemWrite) &&
- ContainsKey(map(), APIPermission::kFileSystemDirectory)) {
- insert(APIPermission::kFileSystemWriteDirectory);
- }
-}
-
PermissionID::PermissionID(APIPermission::ID id)
: std::pair<APIPermission::ID, base::string16>(id, base::string16()) {
}
diff --git a/extensions/common/permissions/api_permission_set.h b/extensions/common/permissions/api_permission_set.h
index 43b97ec..a596a0e 100644
--- a/extensions/common/permissions/api_permission_set.h
+++ b/extensions/common/permissions/api_permission_set.h
@@ -60,8 +60,6 @@ class APIPermissionSet : public BaseSetOperators<APIPermissionSet> {
APIPermissionSet* api_permissions,
base::string16* error,
std::vector<std::string>* unhandled_permissions);
-
- void AddImpliedPermissions();
};
// An ID representing a single permission that belongs to an app or extension.