Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/ext/common.js

Issue 29374674: Issue 4864 - Start using ESLint for adblockpluschrome (Closed)
Patch Set: Fixed typo with shadowRoot getter Created March 14, 2017, 10:28 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/ext/common.js
diff --git a/chrome/ext/common.js b/chrome/ext/common.js
index 5401aaa2885a234c537d2dcbdd4b1267e04f0660..56de7d3960d81ca3f6fb651b4376603f5fbde7e8 100644
--- a/chrome/ext/common.js
+++ b/chrome/ext/common.js
@@ -17,14 +17,16 @@
"use strict";
+(function()
{
// Workaround since HTMLCollection and NodeList didn't have iterator support
// before Chrome 51.
// https://bugs.chromium.org/p/chromium/issues/detail?id=401699
+ let arrayIterator = Array.prototype[Symbol.iterator];
if (!(Symbol.iterator in HTMLCollection.prototype))
- HTMLCollection.prototype[Symbol.iterator] = Array.prototype[Symbol.iterator];
+ HTMLCollection.prototype[Symbol.iterator] = arrayIterator;
if (!(Symbol.iterator in NodeList.prototype))
- NodeList.prototype[Symbol.iterator] = Array.prototype[Symbol.iterator];
+ NodeList.prototype[Symbol.iterator] = arrayIterator;
/* Message passing */
@@ -46,4 +48,4 @@
ext.getURL = chrome.extension.getURL;
ext.i18n = chrome.i18n;
-}
+}());

Powered by Google App Engine
This is Rietveld