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: Use var for ext declarations again Created Feb. 8, 2017, 9:02 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..a058132047784e220906c2b0b8214604386088fc 100644
--- a/ext/common.js
+++ b/ext/common.js
@@ -18,7 +18,9 @@
"use strict";
{
+ /* eslint-disable no-var */
var ext = {};
+ /* eslint-enable no-var */
let EventTarget = ext._EventTarget = function()
{
@@ -36,13 +38,13 @@
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