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: Reduce inline eslint-disable comments Created Feb. 8, 2017, 7:27 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
« no previous file with comments | « chrome/ext/background.js ('k') | chrome/ext/popup.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/ext/common.js
diff --git a/chrome/ext/common.js b/chrome/ext/common.js
index 5401aaa2885a234c537d2dcbdd4b1267e04f0660..3210849c91811e151985d0273893914d25e62965 100644
--- a/chrome/ext/common.js
+++ b/chrome/ext/common.js
@@ -17,33 +17,32 @@
"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];
+// 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] = arrayIterator;
+if (!(Symbol.iterator in NodeList.prototype))
+ NodeList.prototype[Symbol.iterator] = arrayIterator;
- /* Message passing */
+/* Message passing */
- ext.onMessage = new ext._EventTarget();
+ext.onMessage = new ext._EventTarget();
- /* Background page */
+/* Background page */
- ext.backgroundPage = {
- sendMessage: chrome.runtime.sendMessage,
- getWindow()
- {
- return chrome.extension.getBackgroundPage();
- }
- };
+ext.backgroundPage = {
+ sendMessage: chrome.runtime.sendMessage,
+ getWindow()
+ {
+ return chrome.extension.getBackgroundPage();
+ }
+};
- /* Utils */
+/* Utils */
- ext.getURL = chrome.extension.getURL;
- ext.i18n = chrome.i18n;
-}
+ext.getURL = chrome.extension.getURL;
+ext.i18n = chrome.i18n;
« no previous file with comments | « chrome/ext/background.js ('k') | chrome/ext/popup.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld