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

Unified Diff: chrome/browser/extensions/extension_management_api.cc

Issue 7003098: Start refractoring extension permissions into ExtensionPermissionSet. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleanup and update ExtensionPermissionSet data model. Created 9 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/extensions/extension_management_api.cc
diff --git a/chrome/browser/extensions/extension_management_api.cc b/chrome/browser/extensions/extension_management_api.cc
index 0053b1d3a48a60fea7835900faa2f0b9ed91cf00..b67b03f578bd6a6e7164bfadb92ff58e4db31b84 100644
--- a/chrome/browser/extensions/extension_management_api.cc
+++ b/chrome/browser/extensions/extension_management_api.cc
@@ -91,7 +91,8 @@ static DictionaryValue* CreateExtensionInfo(const Extension& extension,
info->Set("icons", icon_list);
}
- const std::set<std::string> perms = extension.api_permissions();
+ const std::set<std::string> perms =
+ extension.permission_set()->GetAPIsAsStrings();
ListValue* permission_list = new ListValue();
if (!perms.empty()) {
std::set<std::string>::const_iterator perms_iter;
@@ -105,7 +106,8 @@ static DictionaryValue* CreateExtensionInfo(const Extension& extension,
ListValue* host_permission_list = new ListValue();
if (!extension.is_hosted_app()) {
// Skip host permissions for hosted apps.
- const URLPatternList host_perms = extension.host_permissions();
+ const URLPatternList host_perms =
+ extension.permission_set()->explicit_hosts().patterns();
if (!host_perms.empty()) {
URLPatternList::const_iterator host_perms_iter;
for (host_perms_iter = host_perms.begin();

Powered by Google App Engine
This is Rietveld 408576698