Bug 1560402 - Backed out changeset 0ad5f1f559f7, not necessary after backout of AM in pref tab. a=backout
authorJorg K <jorgk@jorgk.com>
Mon, 24 Jun 2019 22:18:16 +0200
changeset 35940 0b01d4da30db1a135350792847dbf2f218f97813
parent 35939 ee3c551340ae199379f694d4c7ad90e4d23dee5e
child 35941 fac9fd23d58aca8af9df5ae1faa70b91fa5107af
push id392
push userclokep@gmail.com
push dateMon, 02 Sep 2019 20:17:19 +0000
reviewersbackout
bugs1560402
Bug 1560402 - Backed out changeset 0ad5f1f559f7, not necessary after backout of AM in pref tab. a=backout
mailnews/base/content/msgAccountCentral.js
mailnews/base/prefs/content/AccountManager.js
mailnews/base/prefs/content/accountUtils.js
--- a/mailnews/base/content/msgAccountCentral.js
+++ b/mailnews/base/content/msgAccountCentral.js
@@ -262,17 +262,17 @@ function ComposeAMessage(event) {
 /**
  * Open AccountManager to view settings for a given account
  * @param selectPage  the xul file name for the viewing page,
  *                    null for the account main page, other pages are
  *                    'am-server.xul', 'am-copies.xul', 'am-offline.xul',
  *                    'am-addressing.xul', 'am-smtp.xul'
  */
 function ViewSettings(selectPage) {
-  window.parent.MsgAccountManager(selectPage, selectedServer);
+  window.parent.MsgAccountManager(selectPage);
 }
 
 // Open AccountWizard to create an account
 function CreateNewAccount() {
   window.parent.msgOpenAccountWizard();
 }
 
 function CreateNewAccountTB(type) {
--- a/mailnews/base/prefs/content/AccountManager.js
+++ b/mailnews/base/prefs/content/AccountManager.js
@@ -1499,17 +1499,16 @@ var gAccountTree = {
     }
     // Retrieve the persisted value from XULStore.json.
     // It is stored under the URI of the current document and ID of the XUL element.
     return this._dataStore
                .getValue(document.documentURI, aAccountKey, "open");
   },
 
   _build() {
-    currentAccount = currentPageId = null;
     var bundle = document.getElementById("bundle_prefs");
     function getString(aString) { return bundle.getString(aString); }
     var panels = [{string: getString("prefPanel-server"), src: "am-server.xul"},
                   {string: getString("prefPanel-copies"), src: "am-copies.xul"},
                   {string: getString("prefPanel-synchronization"), src: "am-offline.xul"},
                   {string: getString("prefPanel-diskspace"), src: "am-offline.xul"},
                   {string: getString("prefPanel-addressing"), src: "am-addressing.xul"},
                   {string: getString("prefPanel-junk"), src: "am-junk.xul"}];
--- a/mailnews/base/prefs/content/accountUtils.js
+++ b/mailnews/base/prefs/content/accountUtils.js
@@ -244,17 +244,17 @@ function MsgAccountManager(aSelectPage, 
       let folder = window.GetDefaultAccountRootFolder();
       if (folder instanceof Ci.nsIMsgFolder)
         aServer = folder.server;
     }
   }
 
   if (AppConstants.MOZ_APP_NAME == "thunderbird") {
     // Load the Account Manager in a tab.
-    openPreferencesTab("paneAccount", aServer ? aServer.key : null, { subdialog: aSelectPage });
+    openPreferencesTab("paneAccount", aServer ? aServer.key : null, aSelectPage);
   } else {
     // If the Account Manager is already running, just focus the right server.
     let existingAccountManager = Services.wm.getMostRecentWindow("mailnews:accountmanager");
     if (existingAccountManager) {
       existingAccountManager.focus();
       existingAccountManager.selectServer(aServer);
     } else {
       // Load the Account Manager in a standalone dialog.