484156 - optimize MozAfterPaint a bit

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

In the common case chrome event handler is an element, which is in a
document which has a window. So no need to ask whether it has listeners, 
if we can just ask its window.

Comment 1

[Robert O'Callahan (:roc) (email my personal email if necessary)]

This looks correct, but can you explain why it's a speedup?

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

No need to get and check the event listener manager of the chrome event handler
element.

Currently the checks are
content window -> element(chrome event handler) -> chrome window -> windowroot(top-level-chrome event handler)

With the patch
content window -> chrome window -> windowroot(top-level-chrome event handler)

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

http://hg.mozilla.org/mozilla-central/rev/4af3b0b5888d