937465 - Add a warning when cloneNode/importNode is used without a boolean argument on a node with children

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Boris Zbarsky [:bzbarsky]]

This way maybe we can get b2g app authors to not do it.  Assuming they ever look at warnings...

Comment 1

[Boris Zbarsky [:bzbarsky]]

Of course WarnOnceAbout provides no line/file information, so actually fixing anything based on it kinda sucks.  :(

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: Warn when cloneNode() and importNode() are used without the boolean "deep" argument on nodes that have kids.

Comment 3

[Boris Zbarsky [:bzbarsky]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/adf943f1879e

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/mozilla-central/rev/adf943f1879e

Comment 5

[Kohei Yoshino [:kohei]]

https://developer.mozilla.org/en-US/docs/Web/API/Node.cloneNode
https://developer.mozilla.org/en-US/docs/Web/API/document.importNode
https://developer.mozilla.org/en-US/Firefox/Releases/28/Site_Compatibility