Issue 4871 - Start using ESLint for adblockplusui

ext/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/>.
  */
 
 "use strict";
 
+(function()
 {
   if (typeof ext == "undefined")
     window.ext = {};
 
   function Page(source)
   {
     this._source = source;
   }
   Page.prototype =
   {
@@ skipping 79 matching lines @@
     return catalog;
   };
 
   let selectedLocale = window.navigator.language;
   let match = /[?&]locale=([\w-]+)/.exec(window.location.search);
   if (match)
     selectedLocale = match[1];
 
   let locales = getLocaleCandidates(selectedLocale);
   let catalog = initCatalog(locales[0]);
-  let catalogFile = window.location.pathname.replace(/.*\//, "").

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

Did we agree on placing dots at the end of lines?

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

No, I don't think we agreed anything specific there. Do you have a preference
how this one should look?

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

Only a personal preference of having it at the beginning of the line. I guess
it's up to the author though.

Pro: Less chance of confusion with other statements due to being prefixed with
dot.
Con: Less chance of accidental semi-colon insertion.

[kzar, 2017/03/08 10:29:32]

Done.

-    replace(/\..*/, "") + ".json";
+  let catalogFile = window.location.pathname.replace(/.*\//, "")
+    .replace(/\..*/, "") + ".json";
 
   let replacePlaceholder = function(text, placeholder, content)
   {
     return text.split("$" + placeholder + "$").join(content || "");
   };
 
   let parseMessage = function(rawMessage)
   {
     let text = rawMessage.message;
     let placeholders = [];
@@ skipping 59 matching lines @@
 
         if (locales.length == 0)
           return "";
 
         let locale = locales.shift();
         readCatalog(locale, "common.json");
         readCatalog(locale, catalogFile);
       }
     }
   };
-}
+}());