diff options
author | rdevlin.cronin@chromium.org <rdevlin.cronin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-06 21:47:26 +0000 |
---|---|---|
committer | rdevlin.cronin@chromium.org <rdevlin.cronin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-06 21:47:26 +0000 |
commit | 076ebedaa781461c668ea435ef3af3ca707775d5 (patch) | |
tree | 5189adac73248df281db674fff5ce4c08dda0ce8 /chrome/browser/extensions/api/app_window/app_window_api.cc | |
parent | 4bd4064aede02bd5ae1c0a4b7a01af4830abb47a (diff) | |
download | chromium_src-076ebedaa781461c668ea435ef3af3ca707775d5.zip chromium_src-076ebedaa781461c668ea435ef3af3ca707775d5.tar.gz chromium_src-076ebedaa781461c668ea435ef3af3ca707775d5.tar.bz2 |
Remove deprecated permissions functions from Extension
The methods GetActivePermission() and HasAPIPermission() have been deprecated
for more than a year. Finally remove them.
Also clean up some includes in extension.h, and fix transitive includes.
BUG=379356
TBR=atwilson@chromium.org (c/b/background)
TBR=yurys@chromium.org (c/b/devtools)
TBR=bauerb@chromium.org (c/b/managed_mode)
TBR=dalecurtis@chromium.org (c/b/media)
TBR=dewittj@chromium.org (c/b/notifications)
TBR=msw@chromium.org (c/b/ui)
All TBRs are for strictly mechanical changes - no functional difference.
Review URL: https://codereview.chromium.org/314113010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275531 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/api/app_window/app_window_api.cc')
-rw-r--r-- | chrome/browser/extensions/api/app_window/app_window_api.cc | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/chrome/browser/extensions/api/app_window/app_window_api.cc b/chrome/browser/extensions/api/app_window/app_window_api.cc index 3f05a81..c6623e0 100644 --- a/chrome/browser/extensions/api/app_window/app_window_api.cc +++ b/chrome/browser/extensions/api/app_window/app_window_api.cc @@ -26,6 +26,7 @@ #include "content/public/common/url_constants.h" #include "extensions/browser/extensions_browser_client.h" #include "extensions/browser/image_util.h" +#include "extensions/common/permissions/permissions_data.h" #include "extensions/common/switches.h" #include "third_party/skia/include/core/SkColor.h" #include "ui/base/ui_base_types.h" @@ -225,7 +226,8 @@ bool AppWindowCreateFunction::RunAsync() { return false; if (options->transparent_background.get() && - (GetExtension()->HasAPIPermission(APIPermission::kExperimental) || + (GetExtension()->permissions_data()->HasAPIPermission( + APIPermission::kExperimental) || CommandLine::ForCurrentProcess()->HasSwitch( switches::kEnableExperimentalExtensionApis))) { create_params.transparent_background = *options->transparent_background; @@ -240,7 +242,8 @@ bool AppWindowCreateFunction::RunAsync() { if (options->always_on_top.get()) { create_params.always_on_top = *options->always_on_top.get(); - if (create_params.always_on_top && !GetExtension()->HasAPIPermission( + if (create_params.always_on_top && + !GetExtension()->permissions_data()->HasAPIPermission( APIPermission::kAlwaysOnTopWindows)) { error_ = app_window_constants::kAlwaysOnTopPermission; return false; @@ -419,10 +422,11 @@ bool AppWindowCreateFunction::GetBoundsSpec( AppWindow::Frame AppWindowCreateFunction::GetFrameFromString( const std::string& frame_string) { - if (frame_string == kHtmlFrameOption && - (GetExtension()->HasAPIPermission(APIPermission::kExperimental) || - CommandLine::ForCurrentProcess()->HasSwitch( - switches::kEnableExperimentalExtensionApis))) { + if (frame_string == kHtmlFrameOption && + (GetExtension()->permissions_data()->HasAPIPermission( + APIPermission::kExperimental) || + CommandLine::ForCurrentProcess()->HasSwitch( + switches::kEnableExperimentalExtensionApis))) { inject_html_titlebar_ = true; return AppWindow::FRAME_NONE; } |