Issue 4871 - Start using ESLint for adblockplusui

common.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/* globals Components, E */

[Thomas Greiner, 2017/03/01 17:39:34]

Detail: You're also defining `E` as a local variable via `function E(id)` so do
you even need to specify it here?

[kzar, 2017/03/07 12:48:31]

Well as it is with `window.E = function E(id)` then yes it's somehow required.
But I wonder why we don't simply do `function E(id)` instead, which removes the
need to specify it in the globals comment too. Can you think of any reason to
also assign it to window.E? (Same for the other functions below too.)

[Thomas Greiner, 2017/03/07 13:33:00]

Since we no longer have it inside a closure we no longer need to explicitly
assign it to `window`. Therefore I agree with that change.

+/* globals Components */
 
 "use strict";
 
-window.E = function E(id)
+function E(id)
 {
   return document.getElementById(id);
-};
+}
 
-window.getDocLink = function(link, callback)
+function getDocLink(link, callback)
 {
   ext.backgroundPage.sendMessage({
     type: "app.get",
     what: "doclink",
     link
   }, callback);
-};
+}
 
-window.checkShareResource = function(url, callback)
+function checkShareResource(url, callback)
 {
   ext.backgroundPage.sendMessage({
     type: "filters.blocked",
     url,
     requestType: "SCRIPT",
     docDomain: "adblockplus.org",
     thirdParty: true
   }, callback);
-};
+}
 
-window.openSharePopup = function(url)

[Thomas Greiner, 2017/03/01 17:39:34]

Interesting that we're not using arrow functions for such assignments. Is this
expected?

[kzar, 2017/03/07 12:48:31]

Well I've changed this to a regular named function now, see above.

+function openSharePopup(url)
 {
   let glassPane = E("glass-pane");
   if (!glassPane)
   {
     glassPane = document.createElement("div");
     glassPane.setAttribute("id", "glass-pane");
     document.body.appendChild(glassPane);
   }
 
   let iframe = E("share-popup");
@@ skipping 86 matching lines @@
       glassPane.className = "";
       window.removeEventListener("message", popupMessageListener);
     }
 
     iframe.removeEventListener("load", popupLoadListener);
   };
   iframe.addEventListener("load", popupLoadListener, false);
 
   iframe.src = url;
   glassPane.className = "visible";
-};
+}