Part of bug 880595. Remove unnecessary spell checker initialisations that were causing issues following the reworking of the spell checker to be async. This fixes non-cached compose windows, cached compose windows will still have an issue where inline spell check does not work. r=bwinton,a=bustage fix for CLOSED TREE
authorMark Banner <bugzilla@standard8.plus.com>
Tue, 09 Jul 2013 13:49:36 +0100
changeset 12681 b2a4c19e5f0438fe79c1eeeeda93e8dec7028594
parent 12680 c1021b0b890c5492c07714bd460dbce1b518e1d4
child 12682 bb38043221de534440a9a3996b2022f08c339fbd
push id9313
push userbugzilla@standard8.plus.com
push dateTue, 09 Jul 2013 12:49:50 +0000
treeherdercomm-central@b2a4c19e5f04 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbwinton, bustage
bugs880595
Part of bug 880595. Remove unnecessary spell checker initialisations that were causing issues following the reworking of the spell checker to be async. This fixes non-cached compose windows, cached compose windows will still have an issue where inline spell check does not work. r=bwinton,a=bustage fix for CLOSED TREE
mail/components/compose/content/MsgComposeCommands.js
--- a/mail/components/compose/content/MsgComposeCommands.js
+++ b/mail/components/compose/content/MsgComposeCommands.js
@@ -2339,17 +2339,16 @@ function ComposeStartup(recycled, aParam
       document.getElementById("FormatToolbar").setAttribute("hidden", true);
       document.getElementById("formatMenu").setAttribute("hidden", true);
       document.getElementById("insertMenu").setAttribute("hidden", true);
       document.getElementById("menu_showFormatToolbar").setAttribute("hidden", true);
     }
 
     // Do setup common to Message Composer and Web Composer.
     EditorSharedStartup();
-    InitLanguageMenu();
   }
 
   if (params.bodyIsLink)
   {
     let body = gMsgCompose.compFields.body;
     if (gMsgCompose.composeHTML)
     {
       let cleanBody;
@@ -4798,20 +4797,16 @@ function InitEditor()
   document.getElementById('spellCheckEnable')
           .setAttribute('disabled', !gSpellChecker.canSpellCheck);
   // If canSpellCheck = false, then hidden = false, i.e. show it so that we can
   // still add dictionaries. Else, hide that.
   document.getElementById('spellCheckAddDictionariesMain')
           .setAttribute('hidden', gSpellChecker.canSpellCheck);
   // Then, we enable related UI entries.
   enableInlineSpellCheck(getPref("mail.spellcheck.inline"));
-  let dictMenu = document.getElementById("spellCheckDictionariesMenu");
-  let dictSep = document.getElementById("spellCheckLanguageSeparator");
-  gSpellChecker.addDictionaryListToMenu(dictMenu, dictSep);
-
   gAttachmentNotifier.init(editor.document);
 }
 
 // This function modifies gSpellChecker and updates the UI accordingly. It's
 // called either at startup (see InitEditor above), or when the user clicks on
 // one of the two menu items that allow them to toggle the spellcheck feature
 // (either context menu or Options menu).
 function enableInlineSpellCheck(aEnableInlineSpellCheck)