Issue 301 - Change for each to for .. of .. in lib/

lib/contentPolicy.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2014 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 76 matching lines @@
 
   /**
    * Called on module startup, initializes various exported properties.
    */
   init: function()
   {
     TimeLine.enter("Entered content policy initialization");
 
     // type constant by type description and type description by type constant
     let iface = Ci.nsIContentPolicy;
-    for each (let typeName in contentTypes)
+    for (let typeName of contentTypes)
     {
       if ("TYPE_" + typeName in iface)
       {
         let id = iface["TYPE_" + typeName];
         this.type[typeName] = id;
         this.typeDescr[id] = typeName;
         this.localizedDescr[id] = Utils.getString("type_label_" + typeName.toLowerCase());
       }
     }
 
     this.type.ELEMHIDE = 0xFFFD;
     this.typeDescr[0xFFFD] = "ELEMHIDE";
     this.localizedDescr[0xFFFD] = Utils.getString("type_label_elemhide");
 
     this.type.POPUP = 0xFFFE;
     this.typeDescr[0xFFFE] = "POPUP";
     this.localizedDescr[0xFFFE] = Utils.getString("type_label_popup");
 
-    for each (let type in nonVisualTypes)
+    for (let type of nonVisualTypes)
       this.nonVisual[this.type[type]] = true;
 
     // whitelisted URL schemes
-    for each (let scheme in Prefs.whitelistschemes.toLowerCase().split(" "))
+    for (let scheme of Prefs.whitelistschemes.toLowerCase().split(" "))
       this.whitelistSchemes[scheme] = true;
 
     TimeLine.log("done initializing types");
 
     // Generate class identifier used to collapse node and register corresponding
     // stylesheet.
     TimeLine.log("registering global stylesheet");
 
     let offset = "a".charCodeAt(0);
     for (let i = 0; i < 20; i++)
@@ skipping 204 matching lines @@
 
   /**
    * Asynchronously re-checks filters for given nodes.
    */
   refilterNodes: function(/**Node[]*/ nodes, /**RequestEntry*/ entry)
   {
     // Ignore nodes that have been blocked already
     if (entry.filter && !(entry.filter instanceof WhitelistFilter))
       return;
 
-    for each (let node in nodes)
+    for (let node of nodes)
       Utils.runAsync(refilterNode, this, node, entry);
   }
 };
 Policy.init();
 
 /**
  * Actual nsIContentPolicy and nsIChannelEventSink implementation
  * @class
  */
 let PolicyImplementation =
@@ skipping 15 matching lines @@
     }
     catch (e if e.result == Cr.NS_ERROR_FACTORY_EXISTS)
     {
       // See bug 924340 - it might be too early to init now, the old version
       // we are replacing didn't finish removing itself yet.
       Utils.runAsync(this.init.bind(this));
       return;
     }
 
     let catMan = Utils.categoryManager;
-    for each (let category in this.xpcom_categories)
+    for (let category of this.xpcom_categories)
       catMan.addCategoryEntry(category, this.contractID, this.contractID, false, true);
 
     // http-on-opening-request is new in Gecko 18, http-on-modify-request can
     // be used in earlier releases.
     let httpTopic = "http-on-opening-request";
     if (Services.vc.compare(Utils.platformVersion, "18.0") < 0)
       httpTopic = "http-on-modify-request";
 
     Services.obs.addObserver(this, httpTopic, true);
     Services.obs.addObserver(this, "content-document-global-created", true);
     Services.obs.addObserver(this, "xpcom-category-entry-removed", true);
     Services.obs.addObserver(this, "xpcom-category-cleared", true);
 
     onShutdown.add(function()
     {
       // Our category observers should be removed before changing category
       // memberships, just in case.
       Services.obs.removeObserver(this, httpTopic);
       Services.obs.removeObserver(this, "content-document-global-created");
       Services.obs.removeObserver(this, "xpcom-category-entry-removed");
       Services.obs.removeObserver(this, "xpcom-category-cleared");
 
-      for each (let category in this.xpcom_categories)
+      for (let category of this.xpcom_categories)
         catMan.deleteCategoryEntry(category, this.contractID, false);
 
       // This needs to run asynchronously, see bug 753687
       Utils.runAsync(function()
       {
         registrar.unregisterFactory(this.classID, this);
       }.bind(this));
 
       this.previousRequest = null;
     }.bind(this));
@@ skipping 217 matching lines @@
 }
 
 /**
  * Processes nodes scheduled for post-processing (typically hides them).
  */
 function postProcessNodes()
 {
   let nodes = scheduledNodes;
   scheduledNodes = null;
 
-  for each (let node in nodes)
+  for (let node of nodes)
   {
     // adjust frameset's cols/rows for frames
     let parentNode = node.parentNode;
     if (parentNode && parentNode instanceof Ci.nsIDOMHTMLFrameSetElement)
     {
       let hasCols = (parentNode.cols && parentNode.cols.indexOf(",") > 0);
       let hasRows = (parentNode.rows && parentNode.rows.indexOf(",") > 0);
       if ((hasCols || hasRows) && !(hasCols && hasRows))
       {
         let index = -1;
@@ skipping 102 matching lines @@
   if (!wnd || wnd.closed)
     return;
 
   if (entry.type == Policy.type.OBJECT)
   {
     node.removeEventListener("mouseover", objectMouseEventHander, true);
     node.removeEventListener("mouseout", objectMouseEventHander, true);
   }
   Policy.processNode(wnd, node, entry.type, Utils.makeURI(entry.location), true);
 }