| commit | 6c4b230ca6209b1f349d06344e3d416ed04667f5 | [log] [tgz] |
|---|---|---|
| author | Tim Judkins <[email protected]> | Wed Aug 21 02:00:21 2019 |
| committer | Commit Bot <[email protected]> | Wed Aug 21 02:00:21 2019 |
| tree | 5309b54be30d14cecef584238ee0fc918315c8ad | |
| parent | 286cc4b4e16ab82b46b83207e836d5107d590a8a [diff] [blame] |
[Extensions] Refactor method signatures to not use out params Refactored some of the method signatures in ScriptingPermissionsModifier to use return values instead of out parameters. Also touches the a call to these methods in PermissionsUpdater. Bug: None Change-Id: I5139d8093c02f41a6d4699ce88b030f41f7e8318 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762686 Reviewed-by: Devlin <[email protected]> Commit-Queue: Tim Judkins <[email protected]> Cr-Commit-Position: refs/heads/master@{#688844}
diff --git a/chrome/browser/extensions/permissions_updater.cc b/chrome/browser/extensions/permissions_updater.cc index f1d9f11b..35981d1f 100644 --- a/chrome/browser/extensions/permissions_updater.cc +++ b/chrome/browser/extensions/permissions_updater.cc
@@ -517,9 +517,9 @@ bounded_active = bounded_wrapper.get(); } - std::unique_ptr<const PermissionSet> granted_permissions; - ScriptingPermissionsModifier::WithholdPermissionsIfNecessary( - *extension, *prefs, *bounded_active, &granted_permissions); + std::unique_ptr<const PermissionSet> granted_permissions = + ScriptingPermissionsModifier::WithholdPermissionsIfNecessary( + *extension, *prefs, *bounded_active); if (GetDelegate()) GetDelegate()->InitializePermissions(extension, &granted_permissions);