GitLab

This patch moves IDBRequest::did_throw_in_event_handler_ to
Event::legacy_did_listeners_throw_flag_, which corresponds to
legacyOutputDidListenersThrowFlag in DOM standard:
https://dom.spec.whatwg.org/#concept-event-listener-inner-invoke
(See "Note: The legacyOutputDidListenersThrowFlag is only used by
Indexed Database API.")

There is no behavioral change. This is a small refactoring.

The reason is that it's smarter (and less fragile) to have a
bit flag inside Event to indicate exceptions than letting each
EventTarget maintain the flag. It's guaranteed that each Event
resets the bit flag, no dependency to EventTarget, ability to
detect whether any of listeners throws an exception, etc.

Change-Id: Ie37f9def9e37ac220ea755b1483656f922edfc37
Reviewed-on: https://chromium-review.googlesource.com/1179729Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584375}