203111 - Remove unused class nsXMLExtras

Status: RESOLVED FIXED

(Core :: XML, defect)

Description

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Class nsXMLExtras is not used in nsXMLExtrasModule.cpp and can be removed.

Comment 1

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Actually, it is used to register the XMLExtras components, but we could probably
still get rid of the class and register the components in some other way.

Comment 2

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Attachment: Proposed fix

Noticed nameset class was also unused, so removed that as well.

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 121462 [details] [diff] [review]
Proposed fix

Nice! sr=jst

Comment 4

[harishd]

Comment on attachment 121462 [details] [diff] [review]
Proposed fix

r=harishd

Comment 5

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Trivial, safe, small footprint win. Can I land for 1.4b?

Comment 6

[Asa Dotzler [:asa]]

Comment on attachment 121462 [details] [diff] [review]
Proposed fix

a=asa (on behalf of drivers) for checkin to 1.4b. 

Heikki, use the patch flag "approval1.4b?" to request approval to land a
reviewed fix. Use the bug flag "blocking1.4b?" to let drivers know you think
the release shouldn't happen without this bug being fixed.

Comment 7

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Checked in.