[webkit-reviews] review requested: [Bug 20571] Window object missing Animation/Transition event setter : [Attachment 23078] patch with testcases

[bugzilla-daemon at webkit.org]

Dean Jackson <dino at apple.com> has asked Dave Hyatt <hyatt at apple.com> for
review:
Bug 20571: Window object missing Animation/Transition event setter
https://bugs.webkit.org/show_bug.cgi?id=20571

Attachment 23078: patch with testcases
https://bugs.webkit.org/attachment.cgi?id=23078&action=edit

------- Additional Comments from Dean Jackson <dino at apple.com>
This patch also includes a code cleanup since a lot of this file violated the
coding guidelines and hence offended my eyes. Check the next comment for the
"real" patch.

Also, not all the tests included here are directly relevant to the fix in the
patch, but are the tests that should have been landed with the events in the
first place.

Lastly, there is an outstanding bug that makes using window pretty annoying.
The event dispatch code only fires a transition event if the document was
notified of a listener. If you attach only to Window then the Document doesn't
know, and hence nothing happens. I'll file a separate bug on that. The
workaround is to attach a transition event listener anywhere on the document
(even if it isn't in the right place and is empty).