diff options
| author | rdevlin.cronin <rdevlin.cronin@chromium.org> | 2015-09-24 15:35:49 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-09-24 22:36:35 +0000 |
| commit | e2d0fd0ad14bfd0cd82e0093b1488103af532c8e (patch) | |
| tree | c44c2f971ace4bdfcfcc65df42eb629b74cdc40a /extensions/common/extension_messages.h | |
| parent | 8723a2c2a90e060006d1b45a6ce58bacf17bb9d8 (diff) | |
| download | chromium_src-e2d0fd0ad14bfd0cd82e0093b1488103af532c8e.zip chromium_src-e2d0fd0ad14bfd0cd82e0093b1488103af532c8e.tar.gz chromium_src-e2d0fd0ad14bfd0cd82e0093b1488103af532c8e.tar.bz2 | |
[Extensions] Un-refcount PermissionSet
PermissionSet represents a set of permissions, and for some reason,
it's been refcounted. There's really no reason to have it, and it
makes everything more costly and difficult to reason about. Remove the
refcounting.
Note: This is part 1 of a 2-part series. This removes the ref-counting.
In a followup, I'll go through and update many of the places that
use const PermissionSet* and convert to const &.
BUG=455414
TBR=thestig@chromium.org (misc chrome files with ptr conversion)
TBR=nasko@chromium.org (extension messages - not actually changing any IPC messages)
Review URL: https://codereview.chromium.org/1349613003
Cr-Commit-Position: refs/heads/master@{#350684}
Diffstat (limited to 'extensions/common/extension_messages.h')
| -rw-r--r-- | extensions/common/extension_messages.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extensions/common/extension_messages.h b/extensions/common/extension_messages.h index 4454934..8b81352 100644 --- a/extensions/common/extension_messages.h +++ b/extensions/common/extension_messages.h @@ -249,7 +249,7 @@ struct ExtensionMsg_PermissionSetStruct { const extensions::PermissionSet& permissions); ~ExtensionMsg_PermissionSetStruct(); - scoped_refptr<const extensions::PermissionSet> ToPermissionSet() const; + scoped_ptr<const extensions::PermissionSet> ToPermissionSet() const; extensions::APIPermissionSet apis; extensions::ManifestPermissionSet manifest_permissions; |