Issue 5161 - Use maps and sets where appropriate

ext/common.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2017 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 55 matching lines @@
     sendMessage: chrome.runtime.sendMessage,
     getWindow()
     {
       return chrome.extension.getBackgroundPage();
     }
   };
 
 
   /* Utils */
 
+  ext.mapIterable = function(iterable, callback, thisArg)

[Sebastian Noack, 2017/04/29 22:46:32]

Why do we need this helper? IMO the calling code should just use a for-of loop.

[Manish Jethani, 2017/05/04 14:47:33]

Fair enough, the function that used this now returns an iterable.

Removed.

+  {
+    let iterator = iterable[Symbol.iterator]();
+    return {
+      [Symbol.iterator]()
+      {
+        return this;
+      },
+      next()
+      {
+        let next = iterator.next();
+        if (!next.done)
+          next.value = callback.call(thisArg, next.value);
+        return next;
+      }
+    };
+  };
+
   ext.getURL = chrome.extension.getURL;
   ext.i18n = chrome.i18n;
 }());