diff options
author | vabr@chromium.org <vabr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-27 07:44:24 +0000 |
---|---|---|
committer | vabr@chromium.org <vabr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-27 07:44:24 +0000 |
commit | 04e4bbeda497feac5f7fdebabbb13c885294e253 (patch) | |
tree | 5ea6ffccfd04698a77b1b8f340c8892085c07620 /chrome/browser/extensions/api/declarative_content/content_rules_registry.cc | |
parent | 54e7e7430fe8ded2a77fead9cc436b5d568d540e (diff) | |
download | chromium_src-04e4bbeda497feac5f7fdebabbb13c885294e253.zip chromium_src-04e4bbeda497feac5f7fdebabbb13c885294e253.tar.gz chromium_src-04e4bbeda497feac5f7fdebabbb13c885294e253.tar.bz2 |
Almost all actions in Declarative Web Request require all_urls host permissions
The whitelisted actions are:
* ignoring a rule
* cancelling a request
* redirecting to a blank image or an empty document
+ SendMessageToExtension only checks the request's URL against the host permissions (does not require full permissions).
BUG=145456
Review URL: https://chromiumcodereview.appspot.com/14358004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@196959 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/api/declarative_content/content_rules_registry.cc')
-rw-r--r-- | chrome/browser/extensions/api/declarative_content/content_rules_registry.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/chrome/browser/extensions/api/declarative_content/content_rules_registry.cc b/chrome/browser/extensions/api/declarative_content/content_rules_registry.cc index eb95b47..9589633 100644 --- a/chrome/browser/extensions/api/declarative_content/content_rules_registry.cc +++ b/chrome/browser/extensions/api/declarative_content/content_rules_registry.cc @@ -139,8 +139,12 @@ std::string ContentRulesRegistry::AddRulesImpl( DCHECK(content_rules_.find(rule_id) == content_rules_.end()); scoped_ptr<ContentRule> content_rule( - ContentRule::Create(url_matcher_.condition_factory(), extension_id, - extension_installation_time, *rule, NULL, &error)); + ContentRule::Create(url_matcher_.condition_factory(), + extension_id, + extension_installation_time, + *rule, + ContentRule::ConsistencyChecker(), + &error)); if (!error.empty()) { // Clean up temporary condition sets created during rule creation. url_matcher_.ClearUnusedConditionSets(); |