Chromium Code Reviews| Index: chrome/browser/extensions/active_script_controller_browsertest.cc |
| diff --git a/chrome/browser/extensions/active_script_controller_browsertest.cc b/chrome/browser/extensions/active_script_controller_browsertest.cc |
| index cd607ceb1b661a574202c052cd9e88f45b42cc86..36ca9c773cfba674dacef932c426ad254c74720f 100644 |
| --- a/chrome/browser/extensions/active_script_controller_browsertest.cc |
| +++ b/chrome/browser/extensions/active_script_controller_browsertest.cc |
| @@ -151,8 +151,7 @@ testing::AssertionResult ActiveScriptTester::Verify(Browser* browser) { |
| // script has *not* already executed. |
| // Currently, it's okay for content scripts to execute, because we don't |
| // block them. |
|
not at google - send to devlin
2014/05/15 00:31:54
comment needs updating?
Devlin
2014/05/20 16:48:09
Whoops, thanks.
|
| - // TODO(rdevlin.cronin): Fix this. |
| - if (inject_success_listener_->was_satisfied() && type_ != CONTENT_SCRIPT) { |
| + if (inject_success_listener_->was_satisfied()) { |
| return testing::AssertionFailure() << |
| name_ << "'s script ran without permission."; |
| } |