Issue 4864 - Start using ESLint for adblockpluschrome

lib/io.js

Index: lib/io.js
diff --git a/lib/io.js b/lib/io.js
index 4b352d0036d40e65157666ba728ddd84e2420101..431d91ef19eaf9791a44b0d86234a842ac3425a6 100644
--- a/lib/io.js
+++ b/lib/io.js
@@ -15,6 +15,8 @@
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+/* global FakeFile */

[Sebastian Noack, 2017/02/09 01:04:51]

Is FakeFile used anywhere else? Can't we just turn lib/compat.js into a module?
Same for "Services" which I've seen to be used as global somewhere else.

As far as adblockpluscore is concerned, with the current Adblock Plus for
Firefox on the way out, having this compatibility layer seems wrong to have in
the first place. But for the time being I wouldn't mind to keep it, but at least
we should turn it into a module, to avoid globals in adblockpluschrome, and
assign to window where necessary for compatibility with adblockpluscore.

[kzar, 2017/02/20 10:27:33]

Not that I can see, no.
Well how would that work since lib/compat.js is where we define `require`. Do
you suggest we split the module framework stuff away from the rest of
lib/compat.js and then turn what's left into a module?
IMO it's better to use a global variable and explicitly document it's use with a
ESLint comment than it is to shove the same variable inside the global `window`
Object in order to pretend it doesn't exist. At least then the global state is
documented. (FWIW I agree to reduce global state is almost always a good thing.)

+
 "use strict";
 
 const keyPrefix = "file:";