46624 - Need to move pref "intl.charset.detector" into navigator.properties

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P3)

Description

[Katsuhiko Momoi]

** Observed with 7/26/2000 Win32 build **

The pref item "intl.charset.detector" is still in "all.js" making it difficult to 
localize this value for different locale. 

This value is needed for some languages like Japanese and Korean as the default Auto-Detect
values for browsing.
We should also set "intl.charset.default" at the same time -- used for Editor and Browser.
This item has already been moved to "navigator.properties".

Comment 1

[Frank Tang]

This is related to country picker.
It should be easy to do.
add nsbeta3 polish

Comment 2

[bobj]

I don't  agree this is "polish".  Localizability is not polish.

Comment 3

[Frank Tang]

nsbeta3+ per i18n bug meeting

Comment 4

[Frank Tang]

we need to add intl.charset.detector to navigator.properties
and change the following places
/layout/html/document/src/nsHTMLDocument.cpp, line 169 -- rv = 
prefs->CopyCharPref("intl.charset.detector",
/layout/html/document/src/nsHTMLDocument.cpp, line 709 -- rv_detect = 
pref->CopyCharPref("intl.charset.detector",
/layout/html/document/src/nsHTMLDocument.cpp, line 717 -- 
/layout/xml/document/src/nsXMLDocument.cpp, line 113 -- rv = 
prefs->CopyCharPref("intl.charset.detector",
/layout/xml/document/src/nsXMLDocument.cpp, line 507 -- rv_detect = 
pref->CopyCharPref("intl.charset.detector",
/mailnews/mime/src/comi18n.cpp, line 1276 -- else if 
(NS_SUCCEEDED(prefs->CopyCharPref("intl.charset.detector",
&detector_name))) {
/modules/libpref/src/init/all.js, line 322 -- pref("intl.charset.detector", "");
/xpfe/global/resources/content/charsetOverlay.js, line 157 -- prefvalue = 
pref.CopyCharPref("intl.charset.detector");

Comment 5

[Frank Tang]

change eta to 8/7

Comment 6

[Frank Tang]

remove the origional eta.

Comment 7

[Frank Tang]

fixed and check in. To both commercial and mozilla tree.

Comment 8

[Teruko Kobayashi]

Verified.