Bug 361437 switched the installer to use local GRE by default. With local GRE, various things are missing after installation. Specifically, chardet.dll and unversalchardet.dll, probably others. The symptom people seem to notice is that the "auto-detect" character encoding menu item is missing. The reason for the bustage is that browser.jst deletes the files after they get installed. The upgradeCleanup first deletes files it thinks are not part of the GRE and then files that are part of the GRE /if/ the GRE was not installed locally. Unforunately, the lists are wrong. chardet.dll and universalchardet.dll (among others) are both in the non-GRE list. To fix this, either 1. Identify all files that are part of the GRE (see mozilla/embedding/config/) 2. Move those files from the non-GRE list to the GRE list. or Switch back to non-local GRE.
Neil, can I have your r and sr to back out BenoitRen's patch? And KaiRo's approval? Non-local GRE got much more testing than local.
Created attachment 252897 [details] [diff] [review] Proposed patch By saving the installer files locally and removing the call to upgradeCleanup I was able to verify that only these two files are affected.
Nice. I'd much more prefer if we are able to solve this with keeping locale GRE in place.
Comment on attachment 252897 [details] [diff] [review] Proposed patch right. I just verified that all the other files from gre.xpi were not deleted.
Fix checked in to the trunk, but leaving on the radar for the branch.
Created attachment 253032 [details] [diff] [review] Branch patch Branch really doesn't want chardet.dll but it wants universalchardet.dll (View/Character Coding/Auto-Detect/Universal).
Comment on attachment 253032 [details] [diff] [review] Branch patch a=me for 1.1.1 (given it gets r+)
Comment on attachment 253032 [details] [diff] [review] Branch patch r=me (note that packager/build/ is super-obsolete)
Fixed on the branch too.