Issue 3421 - Get rid of for each loops in keySelector.js

lib/keySelector.js

Index: lib/keySelector.js
===================================================================
--- a/lib/keySelector.js
+++ b/lib/keySelector.js
@@ -90,47 +90,47 @@ KeySelector.prototype =
       if (keyCode && "DOM_" + keyCode.toUpperCase() in Ci.nsIDOMKeyEvent)
         keyData.code = Ci.nsIDOMKeyEvent["DOM_" + keyCode.toUpperCase()];
 
       if (!keyData.char && !keyData.code)
         continue;
 
       let keyModifiers = key.getAttribute("modifiers");
       if (keyModifiers)
-        for each (let modifier in keyModifiers.toUpperCase().match(/\w+/g))
+        for (let modifier of keyModifiers.toUpperCase().match(/\w+/g))
           if (modifier in validModifiers)
             keyData[validModifiers[modifier]] = true;
 
       let canonical = [keyData.shift, keyData.meta, keyData.alt, keyData.control, keyData.char || keyData.code].join(" ");
       this._existingShortcuts[canonical] = true;
     }
   },
 
   /**
    * Selects a keyboard shortcut variant that isn't already taken,
    * parses it into an object.
    */
   selectKey: function(/**String*/ variants) /**Object*/
   {
-    for each (let variant in variants.split(/\s*,\s*/))
+    for (let variant of variants.split(/\s*,\s*/))
     {
       if (!variant)
         continue;
 
       let keyData =
       {
         shift: false,
         meta: false,
         alt: false,
         control: false,
         char: null,
         code: null,
         codeName: null
       };
-      for each (let part in variant.toUpperCase().split(/\s+/))
+      for (let part of variant.toUpperCase().split(/\s+/))
       {
         if (part in validModifiers)
           keyData[validModifiers[part]] = true;
         else if (part.length == 1)
           keyData.char = part;
         else if ("DOM_VK_" + part in Ci.nsIDOMKeyEvent)
         {
           keyData.code = Ci.nsIDOMKeyEvent["DOM_VK_" + part];