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; |