1631439 - Rewrite callers of document.createElementNS(XUL_NS, ...) to use document.createXULElement(...)

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P1)

Description

[Tim Nguyen :ntim]

+++ This bug was initially created as a clone of Bug #1479538 +++

As discussed in https://bugzilla.mozilla.org/show_bug.cgi?id=1479050#c3 and https://bugzilla.mozilla.org/show_bug.cgi?id=1479050#c4, we are planning to add a new chrome-only API called document.createXULElement(el) as sugar around document.createElementNS(XUL_NS, el).

Once we land that, we can migrated existing callers like

• https://searchfox.org/mozilla-central/search?q=createElementNS(XUL_NS&case=false&regexp=false&path=
  And then remove the const XUL_NS declarations (assuming they aren't used anywhere else in scope).

Comment 1

[Tim Nguyen :ntim]

https://searchfox.org/mozilla-central/search?q=createElementNS(XUL_NS%2C&case=false&regexp=false&path=

document.createElementNS(XUL_NS, needs to be replaced with document.createXULElement( in the non-test occurrences only. And the associated XUL_NS definition should be removed (if unused).

Comment 2

[aarushivij]

Hello, Can I take this bug?
Thanks :)
Aarushi

Comment 3

[Tim Nguyen :ntim]

Please do :)

Comment 4

[aarushivij]

Attachment: Bug 1631439 - Rewrite callers of document.createElementNS(XUL_NS, ...) to use document.createXULElement(...) r=ntim

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d363e96d98f5
Rewrite callers of document.createElementNS(XUL_NS, ...) to use document.createXULElement(...) r=ntim

Comment 6

[Noemi Erli[:noemi_erli]]

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