397533 - Remove some useless includes/instance definitions of nsISupportsArray in mailnews.

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

There's some includes of nsISupportsArray.h and some instance definitions for the class that are just redundant and not required. We should remove them and start helping the remove nsISupportsArray effort ;-)

As the content of the interfaces in the idls isn't changing, we don't have to change uuids.

Comment 1

[David :Bienvenu]

Comment on attachment 282299 [details] [diff] [review]
The fix

as long as it still compiles :-)

Comment 2

[Mark Banner (:standard8)]

Checked in -> fixed.

Comment 3

[alta88]

what does this mean for things like, for example, getDragData in nsDragAndDrop.js - will there be some new interface to replace nsISupportsArray functionality?

Comment 4

[Mark Banner (:standard8)]

(In reply to comment #3)
> what does this mean for things like, for example, getDragData in
> nsDragAndDrop.js - will there be some new interface to replace nsISupportsArray
> functionality?

See http://wiki.mozilla.org/Gecko:Obsolete_API and http://mxr.mozilla.org/seamonkey/source/xpcom/ds/nsISupportsArray.idl#42 - nsISupportsArray is obsolete. It will most likely be replaced by an nsIArray interface as thats the most similar.