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

Unified Diff: 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: ext/common.js
diff --git a/ext/common.js b/ext/common.js
index 445dc73ad58621eae3adf7a5170a687e093707bd..097c19dc3866d18c1fbcb6f4a1ced0645ac68b6c 100644
--- a/ext/common.js
+++ b/ext/common.js
@@ -17,9 +17,12 @@
"use strict";
-{
- var ext = {};
+/* eslint-disable no-var */
+var ext = {};
+/* eslint-enable no-var */
Wladimir Palant 2017/03/14 13:03:23 Use `window.ext = {}` inside the IIFE instead of d
kzar 2017/03/15 04:57:44 Done.
+(function()
+{
let EventTarget = ext._EventTarget = function()
{
this._listeners = [];
@@ -36,15 +39,15 @@
if (idx != -1)
this._listeners.splice(idx, 1);
},
- _dispatch()
+ _dispatch(...args)
{
let results = [];
let listeners = this._listeners.slice();
for (let listener of listeners)
- results.push(listener.apply(null, arguments));
+ results.push(listener(...args));
return results;
}
};
-}
+}());

Powered by Google App Engine
This is Rietveld