Revert of Evaluate declarativeContent API rules on add/remove (patchset #3 id:40001...
commit64b78871a088973eb7dc4c36b19509a8526b5a71
authorthakis <thakis@chromium.org>
Sat, 9 May 2015 20:16:42 +0000 (9 13:16 -0700)
committerCommit bot <commit-bot@chromium.org>
Sat, 9 May 2015 20:17:19 +0000 (9 20:17 +0000)
tree7d4c3c2ce52b08cc93f72beef697e88b4b1ef29f
parente82d248552776b6e0aefcdc7157f707735419586
Revert of Evaluate declarativeContent API rules on add/remove (patchset #3 id:40001 of https://codereview.chromium.org/1128323002/)

Reason for revert:
DeclarativeContentApiTest.Overview started to reliably fail on the Mac OS X 10.6 bot after this went in:
http://build.chromium.org/p/chromium.mac/builders/Mac10.6%20Tests/builds/2062
http://build.chromium.org/p/chromium.mac/builders/Mac10.6%20Tests/builds/2063

../../chrome/browser/extensions/api/declarative_content/declarative_content_apitest.cc:134: Failure
Value of: page_action->GetIsVisible(tab_id)
  Actual: true
Expected: false

Original issue's description:
> Evaluate declarativeContent API rules on add/remove
>
> Fixes bug where rules were not being evaluated until tabs' WebContents
> were navigated.
>
> BUG=485172
>
> Committed: https://crrev.com/b6780cb55862276ac886d6acfc6193f885d93f2e
> Cr-Commit-Position: refs/heads/master@{#329048}

TBR=kalman@chromium.org,wittman@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=485172

Review URL: https://codereview.chromium.org/1130793004

Cr-Commit-Position: refs/heads/master@{#329053}
chrome/browser/extensions/api/declarative_content/chrome_content_rules_registry.cc
chrome/browser/extensions/api/declarative_content/chrome_content_rules_registry.h
chrome/browser/extensions/api/declarative_content/chrome_content_rules_registry_unittest.cc
chrome/browser/extensions/api/declarative_content/declarative_content_apitest.cc