diff options
| author | rdevlin.cronin@chromium.org <rdevlin.cronin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-06 05:04:36 +0000 |
|---|---|---|
| committer | rdevlin.cronin@chromium.org <rdevlin.cronin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-06 05:04:36 +0000 |
| commit | a6910e7458ad8ce393b97696307230ceb12a6b77 (patch) | |
| tree | 24afa5b7b8a8c41a24755007e747a67b084d744f /chrome/browser/ui/webui/extensions | |
| parent | bebb1a87602202e572d4a817d5aa047d32a70cf0 (diff) | |
| download | chromium_src-a6910e7458ad8ce393b97696307230ceb12a6b77.zip chromium_src-a6910e7458ad8ce393b97696307230ceb12a6b77.tar.gz chromium_src-a6910e7458ad8ce393b97696307230ceb12a6b77.tar.bz2 | |
Remove PermissionsData::ForExtension() completely
There's no reason to have this. It was originally because PermissionsData
didn't really belong on Extension (and was to be taken off), but now that
PermissionsData is refactored to only include active permissions, it belongs
on the extension object.
TBR=ben@chromium.org (c/b/ui - simple refactor, no functional changes)
TBR=thestig@chromium.org (c/b/media_galleries - simple refactor, no functional changes)
BUG=379356
Review URL: https://codereview.chromium.org/315573003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275332 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/webui/extensions')
| -rw-r--r-- | chrome/browser/ui/webui/extensions/extension_settings_handler.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/chrome/browser/ui/webui/extensions/extension_settings_handler.cc b/chrome/browser/ui/webui/extensions/extension_settings_handler.cc index 9511051..e50de2f 100644 --- a/chrome/browser/ui/webui/extensions/extension_settings_handler.cc +++ b/chrome/browser/ui/webui/extensions/extension_settings_handler.cc @@ -260,9 +260,10 @@ base::DictionaryValue* ExtensionSettingsHandler::CreateExtensionDetailValue( // - The feature is enabled. // - The extension has access to enough urls that we can't just let it run // on those specified in the permissions. - bool wants_all_urls = FeatureSwitch::scripts_require_action()->IsEnabled() && - PermissionsData::ForExtension(extension) - ->RequiresActionForScriptExecution(extension); + bool wants_all_urls = + FeatureSwitch::scripts_require_action()->IsEnabled() && + extension->permissions_data()->RequiresActionForScriptExecution( + extension); extension_data->SetBoolean("wantsAllUrls", wants_all_urls); extension_data->SetBoolean( "allowAllUrls", |