Closed Bug 190826 Opened 22 years ago Closed 21 years ago

nsIRDFNodeList is not used

Categories

(Core :: XUL, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: timeless)

Details

Attachments

(1 file)

nsRDFDOMNodeList implements this interface, but no one uses the interface. We
should just remove it.
Attached patch remove itSplinter Review
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....
Attachment #112773 - Flags: superreview+
Attachment #112773 - Flags: review?(jaggernaut)
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?
Attachment #112773 - Flags: review?(jaggernaut) → review+
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.
Status: NEW → ASSIGNED
fixed
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
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...
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...)
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: shrir → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: