324100 - need to consolidate forced charset setting API

Status: NEW

(MailNews Core :: Internationalization, defect)

Description

[Eyal Rozenberg]

You can do any of the following in the 3-pane window or a message display window:

gLastMailCharset = charsetPref;
gCharsetMenu.SetCurrentMailCharset(charsetPref);
MessengerSetForcedCharacterSet('charset.' + charsetPref);
MessengerSetForcedCharacterSet(charsetPref);
SetDefaultCharacterSet('charset.' + charsetPref);
SetForcedCharset('charset.' + charsetPref);
messenger.SetDocumentCharset(charsetPref);
msgWindow.mailCharacterSet = charsetPref; /* with */ msgWindow.charsetOverride = true;

Some of them have partial effects, some of them complain that the browser is unavailable, but only 

MessengerSetForcedCharacterSet(charsetPref);

is what you really want to be doing to simulate clicking some charset on the charset menu. And this is all very confusing. There needs to be at the very least some documentation regarding all of these, and at best some encapsulation to prevent most of these from being executed without triggering an exception or executed at all.

Comment 1

[Wayne Mery (:wsmwk)]

needed for SM?
Is there a better component for this?
reset QA (was empty)

Comment 2

[Eyal Rozenberg]

> needed for SM?
Yes, I think.