Bug 462423: allow changing and choosing mail start page even when it is not set to show at the start. r=mkmelin

mail/base/content/commandglue.js

@@ -975,15 +975,13 @@ function FolderPaneSelectionChange()
       ClearThreadPane();
     }
 
-    if (gAccountCentralLoaded)
+    var startpageenabled = gPrefBranch.getBoolPref("mailnews.start_page.enabled");
-      UpdateMailToolbar("gAccountCentralLoaded");
+    if (gDisplayStartupPage && startpageenabled)
-
-    if (gDisplayStartupPage)
     {
       loadStartPage();
       gDisplayStartupPage = false;
-      UpdateMailToolbar("gDisplayStartupPage");
     }
+    UpdateMailToolbar("FolderPaneSelectionChange");
 }
 
 function ClearThreadPane()

mail/base/content/mail3PaneWindowCommands.js

@@ -383,7 +383,7 @@ var DefaultController =
           enabled.value = gDBView.navigateStatus((command == "cmd_goBack" || command == "button_goBack") ? nsMsgNavigationType.back : nsMsgNavigationType.forward);
         return enabled.value;
       case "cmd_goStartPage":
-        return pref.getBoolPref("mailnews.start_page.enabled") && !IsMessagePaneCollapsed();
+        return !IsMessagePaneCollapsed();
       case "cmd_markAllRead":
       case "cmd_markReadByDate":
         return IsFolderSelected();

mail/base/content/mailWindow.js

@@ -548,13 +548,14 @@ function loadStartPage()
   try
   {
     gMessageNotificationBar.clearMsgNotifications();
-    var startpageenabled = pref.getBoolPref("mailnews.start_page.enabled");
     var startpage = Components.classes["@mozilla.org/toolkit/URLFormatterService;1"]
                               .getService(Components.interfaces.nsIURLFormatter)
                               .formatURLPref(startPageUrlPref());
 
-    // load about:blank as the start page if we are offline or we don't have a start page url...
+    // Load about:blank as the start page if we are offline or we don't have
-    GetMessagePaneFrame().location.href = startpageenabled && startpage && MailOfflineMgr.isOnline() ? startpage : "about:blank";
+    // a start page url...
+    GetMessagePaneFrame().location.href = startpage && MailOfflineMgr.isOnline() ?
+                                            startpage : "about:blank";
     ClearMessageSelection();
   }
   catch (ex)

mail/components/preferences/general.js

@@ -43,7 +43,6 @@ var gGeneralPane = {
   init: function ()
   {
     this.mPane = document.getElementById("paneGeneral");
-    this.startPageCheck();
     
     this.updatePlaySound();
   },
@@ -87,11 +86,6 @@ var gGeneralPane = {
   },
 #endif
   
-  startPageCheck: function() 
-  {
-    document.getElementById("mailnewsStartPageUrl").disabled = !document.getElementById("mailnewsStartPageEnabled").checked;
-  },
-  
   /**
    * Restores the default start page as the user's start page
    */

mail/components/preferences/general.xul

@@ -88,7 +88,7 @@
       <caption label="&messengerStartPage.label;"/>
         <hbox align="start">
           <checkbox id="mailnewsStartPageEnabled" label="&enableStartPage.label;"
-                    preference="mailnews.start_page.enabled" oncommand="gGeneralPane.startPageCheck();"
+                    preference="mailnews.start_page.enabled"
                     accesskey="&enableStartPage.accesskey;"/>
         </hbox>
         <hbox align="center">