119279 - Changing auto-detector status doesn't reload the message automatically

Status: VERIFIED FIXED

(MailNews Core :: Internationalization, defect)

Description

[ji]

With the current trunk build, turning on auto-detector on mail window doesn't
reload the message.

Steps to reproduce:
1. Launch Mail, and make sure auto-detector is off on Mail window.
2. Have a japanese mail which doesn't have charset info in message body.
3. Turn on auto-detector to Japanese. The message is not reloaded automatically.
The user needs to reload the message manually.

On browser window, changing auto-detector state will reload the page automatically.

Comment 1

[ji]

Nominating for nsbeta1 since it's very inconvenient and confusing to users, and
this problem doesn't exist on browser window.

Comment 2

[nhottanscp]

nsbeta1+ per i18n triage

Comment 3

[rchen]

-> rchen.

Comment 4

[rchen]

Because SelectDetector is set to false in MailMultiplexHandler(event), it won't  
reload the page. After set to true, it will reload just fine.

Naoki, do you know why it was set to false at beginning? 

Comment 5

[rchen]

Attachment: patch for bug fix

Force mailnews to reload the page after user turns on the auto detector.

Comment 6

[rchen]

Naoki, can you review it?

Comment 7

[nhottanscp]

Comment on attachment 66313 [details] [diff] [review]
patch for bug fix

r=nhotta

Comment 8

[rchen]

alecf, can you sr?

Comment 9

[Alec Flett]

Comment on attachment 66313 [details] [diff] [review]
patch for bug fix

sr=alecf

Comment 10

[rchen]

checkin.

Comment 11

[ji]

Verified with 02/04 trunk builds. It's fixed.