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

Unified Diff: extensions/common/extension_messages.cc

Issue 510313002: Extensions-related fixups for scoped_refptr conversion operator removal. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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
« no previous file with comments | « extensions/common/extension_messages.h ('k') | extensions/common/permissions/permissions_data.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/common/extension_messages.cc
diff --git a/extensions/common/extension_messages.cc b/extensions/common/extension_messages.cc
index 129c0ff01c6bed7b0319b6b36a56a3ee58f0a2a3..ce572112d44a6b3a504836bf7a27e74a7992dce6 100644
--- a/extensions/common/extension_messages.cc
+++ b/extensions/common/extension_messages.cc
@@ -26,11 +26,11 @@ ExtensionMsg_PermissionSetStruct::ExtensionMsg_PermissionSetStruct() {
}
ExtensionMsg_PermissionSetStruct::ExtensionMsg_PermissionSetStruct(
- const PermissionSet* permissions)
- : apis(permissions->apis()),
- manifest_permissions(permissions->manifest_permissions()),
- explicit_hosts(permissions->explicit_hosts()),
- scriptable_hosts(permissions->scriptable_hosts()) {
+ const PermissionSet& permissions)
+ : apis(permissions.apis()),
+ manifest_permissions(permissions.manifest_permissions()),
+ explicit_hosts(permissions.explicit_hosts()),
+ scriptable_hosts(permissions.scriptable_hosts()) {
}
ExtensionMsg_PermissionSetStruct::~ExtensionMsg_PermissionSetStruct() {
@@ -53,9 +53,9 @@ ExtensionMsg_Loaded_Params::ExtensionMsg_Loaded_Params(
: manifest(extension->manifest()->value()->DeepCopy()),
location(extension->location()),
path(extension->path()),
- active_permissions(extension->permissions_data()->active_permissions()),
+ active_permissions(*extension->permissions_data()->active_permissions()),
withheld_permissions(
- extension->permissions_data()->withheld_permissions()),
+ *extension->permissions_data()->withheld_permissions()),
id(extension->id()),
creation_flags(extension->creation_flags()) {
}
« no previous file with comments | « extensions/common/extension_messages.h ('k') | extensions/common/permissions/permissions_data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698