invokeEventHandler should take a local handle

invokeEventHandler should take a local handle

V8AbstractEventListener::invokeEventHandler currently takes a v8::Handle
referring to the Event's wrapper. But using this handle in an API call
is unsafe, and appears to be causing crashes (see attached bug).
This patch changes it to take a v8::Local, and updates the callers
to match.

No tests, since there's no way to reliably force GC at the right
time to trigger this crash.

R=jochen@chromium.org
BUG=234760
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=148971

[adamk, 2013-04-23 21:03:50]

Note that I'm not certain this is the culprit, but it definitely looks worth
fixing up invokeEventHandler() to have the right interface in the new world
where everything is v8::Local.

[abarth-chromium, 2013-04-23 21:20:04]

lgtm

[commit-bot: I haz the power, 2013-04-23 21:28:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/adamk@chromium.org/14130021/1

[commit-bot: I haz the power, 2013-04-23 22:29:06]

Retried try job too often on win_layout_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[commit-bot: I haz the power, 2013-04-23 22:37:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/adamk@chromium.org/14130021/1

[commit-bot: I haz the power, 2013-04-23 23:22:45]

Retried try job too often on win_layout_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[commit-bot: I haz the power, 2013-04-24 05:08:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/adamk@chromium.org/14130021/1

[commit-bot: I haz the power, 2013-04-24 05:33:10]

Retried try job too often on win_layout_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[adamk, 2013-04-24 05:36:42]

Committed patchset #1 manually as r148971 (presubmit successful).