Bug 20572 – Window doesn't get Animation/Transition events unless document is listening

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Dean Jackson]

We have code that intercepts event listener registration on the document level to make sure we don't fire unnecessary transition and animation events.  However, if a listener is only on the window it means the event will never fire.

[Simon Fraser (smfr)]

Should the window even get animation events? Is there precedence?

[Dean Jackson]

I did a little research and it seems the window should get these events. However, this is the first time one of the "noisy" events makes sense to be added to the window (others like mutation events aren't turned on). The problem is that the document doesn't register the event type when adding an event listener to the window, the way event target does. It's an easy fix.

[Dean Jackson]

Created attachment 25756 [details]
makes sure Window registers event types

[Dave Hyatt]

Comment on attachment 25756 [details]
makes sure Window registers event types

r=me

[Dean Jackson]

Committed r39037
	M	WebCore/dom/Document.cpp
	M	WebCore/ChangeLog
	M	LayoutTests/ChangeLog
	M	LayoutTests/transitions/transition-end-event-window.html