28854 - clean up intl values in all.js

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P3)

Description

[bobj]

Please remove these obsolete prefs from all.js:

In http://lxr.mozilla.org/seamonkey/source/modules/libpref/src/init/all.js
   
       254 pref("intl.character_set",                  2);     // CS_LATIN1 
       255 pref("intl.font_encoding",                  6);     // CS_MAC_ROMAN
...
       253 pref("intl.auto_detect_encoding",           true);
   

(nhotta, I gave you this because you already have to edit all.jp for bug 28751.)

Comment 1

[nhottanscp]

*** Bug 28751 has been marked as a duplicate of this bug. ***

Comment 2

[nhottanscp]

remove  intl.mailcharset.cyrillic also

Comment 3

[bobj]

Do we need to do anything with obsolete 4.x prefs when doing the profile
migrations?

Comment 4

[nhottanscp]

Asking this to sspitzer, that's a generic pref migration issue.

Comment 5

[(not reading, please use [e-mail] instead)]

there are two parts to this.

1)  remove default 4.x pref values from the default pref files (like all.js) if
they are no longer used by 5.0
2)  migrate 4.x prefs to 5.x values, and then remove the 4.x prefs.

which issue is this?

I have a bug on me (#10672 ) to clear the old 4.x mailnews prefs after we
migrate them to 5.0 pref values. 

Comment 6

[nhottanscp]

This bug is for 1). 
The issue bobj mentioned is 2) which is a dup of 10672.
Please also remove four intl prefs in migration when you do 10672.
intl.character_set
intl.font_encoding
intl.auto_detect_encoding
intl.mailcharset.cyrillic

Comment 7

[(not reading, please use [e-mail] instead)]

who is migrating these prefs?

intl.character_set
intl.font_encoding
intl.auto_detect_encoding
intl.mailcharset.cyrillic

using lxr, I didn't see any code to migrate them.

Comment 8

[nhottanscp]

Those prefs are not going to be migrated to mozilla.
We want those pref not to appear in the migrated pref.
Are they automatically done?

Comment 9

[nhottanscp]

checked in

Comment 10

[Teruko Kobayashi]

Verified.