Issue 4795 - Use modern JavaScript syntax

chrome/ext/common.js

Index: chrome/ext/common.js
diff --git a/chrome/ext/common.js b/chrome/ext/common.js
index 2e5c776dccb493894383117b12bebafc411587b5..127bfb09b4a6f5bb1b9b9ac6604d1a26155d25cd 100644
--- a/chrome/ext/common.js
+++ b/chrome/ext/common.js
@@ -15,8 +15,17 @@
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-(function()
+"use strict";
+
 {
+  // Workaround since HTMLCollection and NodeList didn't have iterator support
+  // before Chrome 51.
+  // https://bugs.chromium.org/p/chromium/issues/detail?id=401699
+  if (!(Symbol.iterator in HTMLCollection.prototype))
+    HTMLCollection.prototype[Symbol.iterator] = Array.prototype[Symbol.iterator];
+  if (!(Symbol.iterator in NodeList.prototype))
+    NodeList.prototype[Symbol.iterator] = Array.prototype[Symbol.iterator];
+
   /* Message passing */
 
   ext.onMessage = new ext._EventTarget();
@@ -26,10 +35,7 @@
 
   ext.backgroundPage = {
     sendMessage: chrome.runtime.sendMessage,
-    getWindow: function()
-    {
-      return chrome.extension.getBackgroundPage();
-    }
+    getWindow() { return chrome.extension.getBackgroundPage(); }

[Sebastian Noack, 2017/01/16 14:47:40]

Turning this into a one-liner seems unrelated. I'm also undecided whether
compressing a function (except for arrow functions) into a single line makes the
the code harder to read. As for consistency, I think we generally avoid that.

[kzar, 2017/01/16 14:59:07]

Done.

   };
 
 
@@ -37,4 +43,4 @@
 
   ext.getURL = chrome.extension.getURL;
   ext.i18n = chrome.i18n;
-})();
+}