#1 Locale uninstallation fails

open
nobody
None
5
2006-08-15
2006-08-15
Motohiko
No

Aviary branch's EM don't handle locales properly.

At installation, EM records information for
uninstallation
(<profileDir>/extensions/<GUID>/uninstall/Uninstall),
but it's incompleted, so cannot uninstall locales.

And, uninstallation method has a bug. It uses
nsIChromeRegistry::deselectLocaleForPackage method, but
it's not exposed.
http://lxr.mozilla.org/aviary101branch/source/content/base/public/nsIChromeRegistry.idl#89

Discussion

  • Motohiko
    Motohiko
    2006-08-15

    Logged In: YES
    user_id=1574141

    This patch directly accesses/overwrites chrome.rdf file when
    uninstall locales. And after uninstallation/unregistration,
    extra restart are added.

     
  • Motohiko
    Motohiko
    2006-08-15

    Attempt to fix

     
    Attachments
  • Motohiko
    Motohiko
    2006-08-20

    Logged In: YES
    user_id=1574141

    Sorry, my fault. Please comment out line 1110.
    - try {
    +// try {

    And for a note:
    Old EM creates wrong uninstallation information file
    (<profileDir>/extensions/<GUID>/uninstall/Uninstall).
    You need to re-install extensions with locales.
    In other words, this patch doesn't have backward compatibility.