242348 - DOMAttrModified does not work if added before insertion

Status: RESOLVED WORKSFORME

(Core :: DOM: Events, defect)

Description

[Erik Arvidsson]

If an element is created using document.createElement and then a mutation event
listener is added to the element before it is inserted into the document the
mutation event is never fired. If another mutation event listener is added (to
any node in the document?) after the node is added all the added event listeners
work.

Comment 1

[Erik Arvidsson]

Attachment: Try modifying the attr and then add another event listener and modify the attr again

Comment 2

[Erik Arvidsson]

Looking at
http://lxr.mozilla.org/mozilla/source/content/events/src/nsEventListenerManager.cpp#665
it seems we are using content.document where we really should use
target.ownerDocument. This might be an issue if the node is never inserted and
we therefore have turned on mutation events, but it should be better than
checking for mutation event listeners when inserting nodes.

(Maybe document = do_QueryInterface(mTarget); is actually returning the
ownerDocument? My Gecko knowledge is pretty limited...)

Comment 3

[Boris Zbarsky [:bzbarsky]]

No, you're right.  We should be using ownerDocument there.  All the more reason
to have GetOwnerDoc() on nsIContent....

Comment 4

[Boris Zbarsky [:bzbarsky]]

This should work now...

Comment 5

[Martijn Wargers (dead)]

Yes, this works now, so this is fixed by bug 27382, right?

Comment 6

[Boris Zbarsky [:bzbarsky]]

Probably fixed by one of the fixes we did to use GetOwnerDoc(), yeah...