705110 - Triggering DOMAttrModified event handler causes "Use of getAttributeNodeNS() is deprecated" warning

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Wladimir Palant]

Attachment: Testcase

I managed to narrow down the source of the "Use of getAttributeNodeNS() is deprecated. Use getAttributeNS() instead." warning that I see occasionally - looks like having a DOMAttrModified event listener around causes that warning if the event listener is triggered. See attached testcase that does nothing other than adding this event listener and triggering it. This is enough to cause this warning to pop up in Error Console. Tested with Firefox 8 and current 11.0a1 nightly on Windows 7 x64.

Comment 1

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

This is kind of ok, since also DOMAttrModified is deprecated.

Need to add a warning about using DOMAttrModified once we have the replacement done.

Comment 2

[Wladimir Palant]

I'm using it because it appears to be the only way to get notified when <xul:tree> finishes editing ("editing" attribute is removed).

Comment 3

[Ian Nartowicz]

DOMAttrModified deprecated?  Maybe there needs to be a replacement first?

Comment 4

[Stefan Sitter [:ssitter]]

The Lightning calendaring extension for Thunderbird triggers the same warning because it is using DOMAttrModified too (http://mxr.mozilla.org/comm-central/search?string=DOMAttrModified&find=/calendar/)

When is it going to be removed? What should be used instead?

Comment 5

[Masatoshi Kimura [:emk]]

Attachment: Stop warning spam

We shouldn't spam the warning until the replacement is available. Even if we want some kinds of warnings, the current message does not indicate what actually cause the warning.

Comment 6

[Mozilla RelEng Bot]

Autoland Patchset:
	Patches: 600100
	Branch: mozilla-central => try
	Destination: http://hg.mozilla.org/try/pushloghtml?changeset=f2ffdd8bfda3
Try run started, revision f2ffdd8bfda3. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=f2ffdd8bfda3

Comment 7

[Mozilla RelEng Bot]

Try run for f2ffdd8bfda3 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=f2ffdd8bfda3
Results (out of 215 total builds):
    exception: 4
    success: 175
    warnings: 21
    failure: 15
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-f2ffdd8bfda3

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/547e2b0fbe83

Comment 9

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/547e2b0fbe83