496429 - Clean up /mailnews/mailnews.js

Status: NEW

(MailNews Core :: Backend, defect)

Description

[Karsten Düsterloh]

With bug 390262 fixed, many of the default prefs defined in /mailnews/mailnews.js are used solely outside of mailnews, eg. in UI code specific to SM or TB. Some of these are even already overwritten in SM's browser-prefs.js (sic!) or in TB's all-thunderbird.js...

I'd propose keeping only those prefs in /mailnews/mailnews.js which are actually used by code under /mailnews and moving everything else into /suite/mailnews/mailnews.js and /mail/app/profile/all-thunderbird.js, respectively.

Comments?

Comment 1

[Mark Banner (:standard8)]

What will you do with prefs that are different in SM versus TB, but are used by mailnews/ code?

Otherwise I think its a great idea.

Comment 2

[Karsten Düsterloh]

(In reply to comment #1)
> What will you do with prefs that are different in SM versus TB, but are used by
> mailnews/ code?

There's no "best" solution in this case, so I'd just keep their current definitions and overwritten values where they are (modulo moving any overwrites out of browser-prefs.js in /suite).