1434819 - Remove XPCOM DOM traversal interfaces

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P2)

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: part 1. Remove unused methods from nsIDOMNodeIterator

MozReview-Commit-ID: 6mVDmDNxx4p

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: part 2. Remove nsIDOMNodeIterator

MozReview-Commit-ID: 7ln10gj1cLP

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: part 3. Stop using XPCOM treewalker bits in webbrowserpersist

MozReview-Commit-ID: GmY7L4p00D1

Comment 4

[Boris Zbarsky [:bzbarsky]]

Attachment: part 4. Remove all the methods from nsIDOMTreeWalker

MozReview-Commit-ID: BDxOLArlCYb

Comment 5

[Boris Zbarsky [:bzbarsky]]

Attachment: part 5. Remove nsIDOMTreeWalker

MozReview-Commit-ID: QmxoxVhppG

Comment 6

[Boris Zbarsky [:bzbarsky]]

Attachment: part 6. Get rid of NodeFilterHolder

Now that TreeWalker and NodeIterator only use webidl APIs, they always have a
NodeFilter, not nsIDOMNodeFilter.

MozReview-Commit-ID: 5pzn9V3Kxzy

Comment 7

[Boris Zbarsky [:bzbarsky]]

Attachment: part 7. Remove the SHOW_* constants from nsIDOMNodeFilter

MozReview-Commit-ID: 7E2KZkLfbSI

Comment 8

[Boris Zbarsky [:bzbarsky]]

Attachment: part 8. Remove the FILTER_* constants from nsIDOMNodeFilter

MozReview-Commit-ID: LSDXZVGDtBc

Comment 9

[Boris Zbarsky [:bzbarsky]]

Attachment: part 9. Remove nsIDOMNodeFilter

MozReview-Commit-ID: JsxGp9HQ2ak

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4e46d45e3fb7
part 1.  Remove unused methods from nsIDOMNodeIterator.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/dd777984656c
part 2.  Remove nsIDOMNodeIterator.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/fe13d3d55123
part 3.  Stop using XPCOM treewalker bits in webbrowserpersist.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/09156c683ba7
part 4.  Remove all the methods from nsIDOMTreeWalker.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/bba7d4e476bb
part 5.  Remove nsIDOMTreeWalker.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/6a93f9c7d0f3
part 6.  Get rid of NodeFilterHolder.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/062e1d691fc5
part 7.  Remove the SHOW_* constants from nsIDOMNodeFilter.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/8ad255c6fd62
part 8.  Remove the FILTER_* constants from nsIDOMNodeFilter.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/bffbab07fdc4
part 9.  Remove nsIDOMNodeFilter.  r=qdot

Comment 11

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Adding this as a dependency of Bug 1387178, since we need to establish telemetry markers for seeing how many websites use Ci.nsINodeFilter.

Comment 12

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/4e46d45e3fb7
https://hg.mozilla.org/mozilla-central/rev/dd777984656c
https://hg.mozilla.org/mozilla-central/rev/fe13d3d55123
https://hg.mozilla.org/mozilla-central/rev/09156c683ba7
https://hg.mozilla.org/mozilla-central/rev/bba7d4e476bb
https://hg.mozilla.org/mozilla-central/rev/6a93f9c7d0f3
https://hg.mozilla.org/mozilla-central/rev/062e1d691fc5
https://hg.mozilla.org/mozilla-central/rev/8ad255c6fd62
https://hg.mozilla.org/mozilla-central/rev/bffbab07fdc4

Comment 13

[Oriol Brufau [:Oriol]]

https://addons.mozilla.org/en-US/firefox/addon/tab-counter/ uses Ci.nsIDOMNodeFilter.SHOW_TEXT
Now you removed it, NodeFilter is not defined in that context, Cu.importGlobalProperties can't import it, and the tree walker is created in a document with no defaultView.
So what I am supposed to do other than hardcode 4?
Well, I guess I can always create a windowless browser, but it seems so ugly:
Services.appShell.createWindowlessBrowser().document.defaultView.NodeFilter.SHOW_TEXT
Better ideas?

Comment 14

[Jorg K (CEST = GMT+2)]

Slightly better idea:
https://hg.mozilla.org/comm-central/rev/a1956e0c9e16d8c4f5ceb7266534829af0de03b5#l1.15

Comment 15

[Boris Zbarsky [:bzbarsky]]

In that situation, I hardcoded 4 with a comment that it's NodeFilter.SHOW_TEXT.

Though if this is being enough of a problem, we can make Cu.importGlobalProperties work for NodeFilter.  It's pretty simple to do.

Comment 16

[Oriol Brufau [:Oriol]]

(In reply to Boris Zbarsky [:bz] (no decent commit message means r-) from comment #15)
> Though if this is being enough of a problem, we can make
> Cu.importGlobalProperties work for NodeFilter.  It's pretty simple to do.

This would be useful, thanks. One can use Cu.importGlobalProperties to import XMLHttpRequest,
so I think it should also be possible to import NodeFilter in order to create a tree walker
in the resulting responseXML.

In fact I think that Cu.importGlobalProperties should be able to import any global property,
I don't understand why the list is so limited.

Comment 17

[Boris Zbarsky [:bzbarsky]]

> I don't understand why the list is so limited.

Because many DOM things don't work properly in a JSM.  The ones that are available are the ones that have been either audited or fixed to ensure they _do_ work there.

Comment 18

[Boris Zbarsky [:bzbarsky]]

I filed bug 1435764 on adding NodeFilter to importGlobalProperties.

Comment 19

[Boris Zbarsky [:bzbarsky]]

Bug 1435764 landed, but...  How is https://addons.mozilla.org/en-US/firefox/addon/tab-counter/ using Ci.nsIDOMNodeFilter?  I thought we didn't expose that stuff to addons anymore.  And if that addon only works with pre-57 Firefox versions, this fix would seem irrelevant to it...

Comment 20

[Oriol Brufau [:Oriol]]

Yes, it's a legacy add-on. With the appropriate about:config flags I can run it on firefox 60. It just needed some small fixes because of bug 437381, bug 1098412 and bug 1319512. With bug 1435764 I will be able to fix this properly, thanks.