515397 - Move messengerdnd.js to suite/

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Rain [:rain1] (inactive)]

We don't use it any more.

<http://mxr.mozilla.org/comm-central/search?find=%2F&string=messengerdnd.js>

Comment 1

[Phil Ringnalda (:philor)]

Attachment: mv

Not sure who'll let me land this when, but it's both harmless and mostly harmless to not land.

Comment 2

[Robert Kaiser]

Actually, I feel somewhat ambiguous about this. For one thing, it would be helpful if you told us that bug 414038 obsoleted it for Thunderbird, but now that I dug that out, I wonder if bug 507601 might actually make it for 2.0 and therefore obsolete and hopefully remove this file altogether.

Comment 3

[Serge Gautherie (:sgautherie)]

messenger.xul occurrence: it is expected to be removed by bug 510793.
SearchDialog.xul occurrence: I noticed it but I haven't look at it yet.

jar.mn: this patch seems safe/fine to do in the meantime.

Comment 4

[Robert Kaiser]

Comment on attachment 399502 [details] [diff] [review]
mv

OK, given Serge, who's doing the work on porting the stuff that obsoleted it on Thunderbird's side, is aware of it and says it's safe for his work, we can do it. I just don't want suite/ to be the collection point for obsolete code ;-)

Comment 5

[Serge Gautherie (:sgautherie)]

(In reply to comment #3)
> SearchDialog.xul occurrence: I noticed it but I haven't look at it yet.

Bug 500004 removed it from TB. (I'll discuss SM port there...)

Comment 6

[Phil Ringnalda (:philor)]

(In reply to comment #4)
> I just don't want suite/ to be the collection point for obsolete code ;-)

Nah, that's xpfe/ :)

Comment 7

[Phil Ringnalda (:philor)]

Comment on attachment 399502 [details] [diff] [review]
mv

Not quite sure what approval flags I want, so I'll just spam 'em.

Comment 8

[Phil Ringnalda (:philor)]

http://hg.mozilla.org/comm-central/rev/5be3378df279