Issue 4864 - Start using ESLint for adblockpluschrome

lib/utils.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
@@ skipping 19 matching lines @@
       // Make sure to not run asynchronous actions before all
       // scripts loaded. This caused issues on Opera in the past.
       let onDOMContentLoaded = () =>
       {
         document.removeEventListener("DOMContentLoaded", onDOMContentLoaded);
         callback();
       };
       document.addEventListener("DOMContentLoaded", onDOMContentLoaded);
     }
     else
-    {
       setTimeout(callback, 0);
-    }
   },
   get appLocale()
   {
     let locale = ext.i18n.getMessage("@@ui_locale").replace(/_/g, "-");
     Object.defineProperty(this, "appLocale", {value: locale, enumerable: true});
     return this.appLocale;
   },
   generateChecksum(lines)
   {
     // We cannot calculate MD5 checksums yet :-(
     return null;
   },
   checkLocalePrefixMatch(prefixes)
   {
     if (!prefixes)
       return null;
 
     for (let prefix of prefixes.split(","))
+    {
       if (new RegExp("^" + prefix + "\\b").test(this.appLocale))
         return prefix;
+    }
 
     return null;
   },
 
   chooseFilterSubscription(subscriptions)
   {
     let selectedItem = null;
     let selectedPrefix = null;
     let matchCount = 0;
     for (let subscription of subscriptions)
     {
       if (!selectedItem)
         selectedItem = subscription;
 
-      let prefix = Utils.checkLocalePrefixMatch(subscription.getAttribute("prefixes"));
+      let prefix = Utils.checkLocalePrefixMatch(
+        subscription.getAttribute("prefixes")
+      );
       if (prefix)
       {
         if (!selectedPrefix || selectedPrefix.length < prefix.length)
         {
           selectedItem = subscription;
           selectedPrefix = prefix;
           matchCount = 1;
         }
         else if (selectedPrefix && selectedPrefix.length == prefix.length)
         {
@@ skipping 10 matching lines @@
           }
         }
       }
     }
     return selectedItem;
   },
 
   getDocLink(linkID)
   {
     let docLink = require("prefs").Prefs.documentation_link;
-    return docLink.replace(/%LINK%/g, linkID).replace(/%LANG%/g, Utils.appLocale);
+    return docLink.replace(/%LINK%/g, linkID).replace(/%LANG%/g,
+                                                      Utils.appLocale);

[Wladimir Palant, 2017/03/14 13:03:29]

I'd rather put the entire .replace on the next line and align the dots.

[kzar, 2017/03/15 04:57:52]

Done.

   },
 
   yield()
   {
   }
 };