Bug 1286930 - Allow the About dialog to show the application update UI even if the separate Update Available window is shown. r?rstrong draft
authorJared Wein <jwein@mozilla.com>
Thu, 14 Jul 2016 15:22:11 -0400
changeset 387727 ea8f1386d60d5cbb60c51588e06b8c43cc67e44b
parent 385190 b7c937889aacde63e7fef321b58d77b5cc31a149
child 525437 2eca8d5442427c54e452a35e7910eca227b070ee
push id23057
push userjwein@mozilla.com
push dateThu, 14 Jul 2016 19:22:27 +0000
reviewersrstrong
bugs1286930
milestone50.0a1
Bug 1286930 - Allow the About dialog to show the application update UI even if the separate Update Available window is shown. r?rstrong MozReview-Commit-ID: 1jUyLx4KsZG
browser/base/content/aboutDialog-appUpdater.js
--- a/browser/base/content/aboutDialog-appUpdater.js
+++ b/browser/base/content/aboutDialog-appUpdater.js
@@ -23,19 +23,22 @@ function onUnload(aEvent) {
   gAppUpdater = null;
 }
 
 
 function appUpdater()
 {
   this.updateDeck = document.getElementById("updateDeck");
 
-  // Hide the update deck when there is already an update window open to avoid
-  // syncing issues between them.
-  if (Services.wm.getMostRecentWindow("Update:Wizard")) {
+  // Hide the update deck when the update window is already open and it's not
+  // already applied, to avoid syncing issues between them. Applied updates
+  // don't have any information to sync between the windows as they both just
+  // show the "Restart to continue"-type button.
+  if (Services.wm.getMostRecentWindow("Update:Wizard") &&
+      !this.isApplied) {
     this.updateDeck.hidden = true;
     return;
   }
 
   XPCOMUtils.defineLazyServiceGetter(this, "aus",
                                      "@mozilla.org/updates/update-service;1",
                                      "nsIApplicationUpdateService");
   XPCOMUtils.defineLazyServiceGetter(this, "checker",