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

Unified Diff: chrome/common/extensions/extension_messages.cc

Issue 51433002: Enable permission warnings from ManifestHandlers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Working on adding ManifestPermissionSet to PermissionSet. Created 7 years, 1 month 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/common/extensions/extension_messages.cc
diff --git a/chrome/common/extensions/extension_messages.cc b/chrome/common/extensions/extension_messages.cc
index 0e3c5e4501a1407ab7cdcf997c730c3cbb43216c..dace46d4a7c178760d8946f9d110bd5ea1fb9deb 100644
--- a/chrome/common/extensions/extension_messages.cc
+++ b/chrome/common/extensions/extension_messages.cc
@@ -32,6 +32,8 @@ ExtensionMsg_Loaded_Params::ExtensionMsg_Loaded_Params(
location(extension->location()),
path(extension->path()),
apis(extension->GetActivePermissions()->apis()),
+ manifest_permissions(
+ extension->GetActivePermissions()->manifest_permissions()),
explicit_hosts(extension->GetActivePermissions()->explicit_hosts()),
scriptable_hosts(extension->GetActivePermissions()->scriptable_hosts()),
id(extension->id()),
@@ -45,7 +47,8 @@ scoped_refptr<Extension> ExtensionMsg_Loaded_Params::ConvertToExtension(
if (extension.get()) {
extensions::PermissionsData::SetActivePermissions(
extension.get(),
- new PermissionSet(apis, explicit_hosts, scriptable_hosts));
+ new PermissionSet(apis, manifest_permissions,
+ explicit_hosts, scriptable_hosts));
}
return extension;
}

Powered by Google App Engine
This is Rietveld 408576698