190826 - nsIRDFNodeList is not used

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[Boris Zbarsky [:bzbarsky]]

nsRDFDOMNodeList implements this interface, but no one uses the interface. We
should just remove it.

Comment 1

[timeless]

Attachment: remove it

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 112773 [details] [diff] [review]
remove it

sr=me.	Jag, would you review?

I suspect that if we manage to switch over to content lists for the
GetElementsBy* functions we can remove this whole class....

Comment 3

[jag (Peter Annema)]

Comment on attachment 112773 [details] [diff] [review]
remove it

So Mozilla isn't using this, and by removing this we're saying it's not
something we expect others to ever (want to) use?

Comment 4

[timeless]

mozilla and any public published consumers don't use the interface. The two
methods are used by the only consumer but they aren't relying on xpcom.

Comment 5

[timeless]

fixed

Comment 6

[Boris Zbarsky [:bzbarsky]]

Any idea why Tp went up at about the same time?  I don't see why this would have
mattered to Tp, but there were no other changes at that time...

Comment 7

[timeless]

sorry, nope.

i'd love to hope it's related to a bad dependency or some other randomness
(we use live sites, not copies, right?)

anyway, here's an alternate view:

	Before		Max	Min	After		
IDX	MAX	MIN	delta	delta	MAX	MIN	PATH
2	1123	761	132	12	1255	773	www.aol.com
8	867	701	-35	4	832	705	www.compuserve.com
9	880	545	-43	5	837	550	www.msnbc.com
19	2948	2435	49	27	2997	2462	www.voodooextreme.com
26	1488	1067	86	9	1574	1076	www.zdnet.com_Gamespot.com
28	676	510	-30	4	646	514	www.microsoft.com
37	1592	1420	40	11	1632	1431	www.cnn.com`

note that I don't have experience playing with these numbers, so my
manipulations might not be valid.

Actually they aren't very useful (voodooextreme has a negative average delta...)