[Extensions Bindings] Remove ScriptContext::DispatchEvent
Remove ScriptContext::DispatchEvent. In JS bindings, this would just
call Event.dispatchEvent with passed in v8 arguments; however, this
won't work in the native bindings system (where we don't want to support
this type of direct call into JS).
Remove the method on ScriptContext, and update callers to instead use
ExtensionBindingsSystem::DispatchEventInContext() (which for JS
bindings, behaves the same way, but does the right thing for native
bindings). In a few cases, this is a little unfortunate, since it
results in extra conversion to base::Values before converting to v8
arguments in the event; however, this is relatively little work, done
in very few places, and results in a much simpler design.
BUG=653596
Review-Url: https://codereview.chromium.org/2770403002
Cr-Commit-Position: refs/heads/master@{#460389}
14 files changed
