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=Standard8 and bustage fix for CLOSED TREE
authorMark Banner <bugzilla@standard8.plus.com>
Tue, 09 Jul 2013 13:49:36 +0100
changeset 14176 7f804e500286e5fe8a8ab698ce07bbd13ff8ec92
parent 14175 3777b122c29dc5f1459f6c2d0ac906fb73a834a6
child 14177 496529ef862ed62e75d71e7371271bad3cd028b9
push id975
push userbugzilla@standard8.plus.com
push dateTue, 09 Jul 2013 16:25:34 +0000
treeherdercomm-aurora@7f804e500286 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbwinton, Standard8
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=Standard8 and 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)