Bug 415175 - Updating to an extension where the updated xpi requires a compatibility update fails. r=robstrong.
authordtownsend@oxymoronical.com
Tue, 11 Mar 2008 16:35:21 -0700
changeset 12906 0e48fbc9658d40bd0e65484a6cd169b10912d555
parent 12905 63bced81ce38fc920aa5c5fac3cce9d35a235e33
child 12907 08c95a0d7453854fda51addc1bd0b188d8ca3058
push idunknown
push userunknown
push dateunknown
reviewersrobstrong
bugs415175
milestone1.9b5pre
Bug 415175 - Updating to an extension where the updated xpi requires a compatibility update fails. r=robstrong.
toolkit/mozapps/extensions/src/nsExtensionManager.js.in
--- a/toolkit/mozapps/extensions/src/nsExtensionManager.js.in
+++ b/toolkit/mozapps/extensions/src/nsExtensionManager.js.in
@@ -2673,18 +2673,16 @@ ExtensionManager.prototype = {
 
   /**
    * See nsIExtensionManager.idl
    */
   start: function(commandLine) {
     var isDirty = false;
     var forceAutoReg = false;
 
-    this._showUpdatesWindow();
-
     // Somehow the component list went away, and for that reason the new one
     // generated by this function is going to result in a different compreg.
     // We must force a restart.
     var componentList = getFile(KEY_PROFILEDIR, [FILE_EXTENSION_MANIFEST]);
     if (!componentList.exists())
       forceAutoReg = true;
 
     // Check for missing manifests - e.g. missing extensions.ini, missing
@@ -2696,16 +2694,18 @@ ExtensionManager.prototype = {
 
     // Configure any items that are being installed, uninstalled or upgraded
     // by being added, removed or modified by another process. We must do this
     // on every startup since there is no way we can tell if this has happened
     // or not!
     if (this._checkForFileChanges())
       isDirty = true;
 
+    this._showUpdatesWindow();
+
     if (PendingOperations.size != 0)
       isDirty = true;
 
     // Extension Changes
     if (isDirty) {
       var needsRestart = this._finishOperations();
 
       if (forceAutoReg) {