28863 - [FEATURE] default charset for message view

Status: VERIFIED FIXED

(MailNews Core :: MIME, defect, P3)

Description

[nhottanscp]

The default charset is needed to read mails without charset specified. 
I think this can be done by defining a pref and set it to default_charset in 
libimime.
Proposed pref name,
mailnews.view_default_charset (we already have mailnews.send_default_charset)

Comment 1

[rhp (gone)]

Ok, I think that I have this working in my tree. Naoki, can you send me a 
test mail message that I can verify that it is working correctly.

Thanks.

- rhp

Comment 2

[nhottanscp]

Attachment: zipped local mail folder contains a mail with no charset label

Comment 3

[nhottanscp]

I don't have a real mail with no charset label, momoi san do you have any?
I modified one of the smoke test message, removed a charset label manually. That 
mail cannot be shown because no charset label.

Comment 4

[Katsuhiko Momoi]

I can send you raw a ISO-2022-JP mail message in both headers 
and text body.
Send me a msg indicating where you want a test msg sent. I'll
send back a msg without any MIME charset.

Comment 5

[rhp (gone)]

You can send me a message at [e-mail]

- rhp

Comment 6

[rhp (gone)]

I think I have this fixed...Naoki has a patch. - rhp

Comment 7

[nhottanscp]

Need investigation of this behavior by i18n, reassing to nhotta.

Comment 8

[nhottanscp]

Default charset is applied to main body correctly with Rich's latest patch.
We still have header display issue but that can be treated as separately.
Reassign back to rhp.

Comment 9

[rhp (gone)]

The hidden pref is now enabled.

- rhp

Comment 10

[Katsuhiko Momoi]

** Observed with 3/6/2000 Win32 build **

This feature seems to be working well. I can view unlabeled
Japanese message body without doing anything. I even ventured out
to the wild world of Russian and Bulgarian newsgroups with
the default vew charset set to KOI8-R and Windows-1251, respectively.
The results were very nice -- Communicator 4.x was never this good.
I can read unlabeled newsgroup articles which seem to be the rule
rather than exceptions in these groups. The default view charset
is definitely needed! 

Marking it verified as fixed.