| Index: extensions/common/permissions/permissions_data.h
|
| diff --git a/extensions/common/permissions/permissions_data.h b/extensions/common/permissions/permissions_data.h
|
| index ba2f3c3aa54078bbca0e1843e7fc2ec9942e3e70..74af40e9d10f078649609a3cd4c53148995b5e4b 100644
|
| --- a/extensions/common/permissions/permissions_data.h
|
| +++ b/extensions/common/permissions/permissions_data.h
|
| @@ -115,15 +115,16 @@ class PermissionsData {
|
| APIPermission::ID permission,
|
| const APIPermission::CheckParam* param) const;
|
|
|
| - // TODO(rdevlin.cronin): GetEffectiveHostPermissions(), HasHostPermission(),
|
| - // and HasEffectiveAccessToAllHosts() are just forwards for the active
|
| + // Returns the effective hosts associated with the active permissions and the
|
| + // current tab permissions.
|
| + URLPatternSet GetEffectiveHostPermissions() const;
|
| +
|
| + // TODO(rdevlin.cronin): HasHostPermission() and
|
| + // HasEffectiveAccessToAllHosts() are just forwards for the active
|
| // permissions. We should either get rid of these, and have callers use
|
| // active_permissions(), or should get rid of active_permissions(), and make
|
| // callers use PermissionsData for everything. We should not do both.
|
|
|
| - // Returns the effective hosts associated with the active permissions.
|
| - const URLPatternSet& GetEffectiveHostPermissions() const;
|
| -
|
| // Whether the extension has access to the given |url|.
|
| bool HasHostPermission(const GURL& url) const;
|
|
|
|
|