Issue 4878 - Start using ESLint for adblockpluscore

lib/synchronizer.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 URL, XMLHttpRequest */
-
 "use strict";
 
 /**
  * @fileOverview Manages synchronization of filter subscriptions.
  */
 
-Cu.import("resource://gre/modules/XPCOMUtils.jsm");
-Cu.import("resource://gre/modules/Services.jsm");
-
-let {Downloader, Downloadable,
-    MILLIS_IN_SECOND, MILLIS_IN_MINUTE,
-    MILLIS_IN_HOUR, MILLIS_IN_DAY} = require("downloader");
-let {Filter, CommentFilter} = require("filterClasses");
-let {FilterStorage} = require("filterStorage");
-let {FilterNotifier} = require("filterNotifier");
-let {Prefs} = require("prefs");
-let {Subscription, DownloadableSubscription} = require("subscriptionClasses");
-let {Utils} = require("utils");
-
-let INITIAL_DELAY = 1 * MILLIS_IN_MINUTE;
-let CHECK_INTERVAL = 1 * MILLIS_IN_HOUR;
-let DEFAULT_EXPIRATION_INTERVAL = 5 * MILLIS_IN_DAY;
+const {Downloader, Downloadable,
+       MILLIS_IN_SECOND, MILLIS_IN_MINUTE,
+       MILLIS_IN_HOUR, MILLIS_IN_DAY} = require("downloader");
+const {Filter} = require("filterClasses");
+const {FilterStorage} = require("filterStorage");
+const {FilterNotifier} = require("filterNotifier");
+const {Prefs} = require("prefs");
+const {Subscription, DownloadableSubscription} = require("subscriptionClasses");
+const {Utils} = require("utils");
+
+const INITIAL_DELAY = 1 * MILLIS_IN_MINUTE;
+const CHECK_INTERVAL = 1 * MILLIS_IN_HOUR;
+const DEFAULT_EXPIRATION_INTERVAL = 5 * MILLIS_IN_DAY;
 
 /**
  * The object providing actual downloading functionality.
  * @type {Downloader}
  */
 let downloader = null;
 
 /**
  * This object is responsible for downloading filter subscriptions whenever
  * necessary.
@@ skipping 14 matching lines @@
     });
 
     downloader.onExpirationChange = this._onExpirationChange.bind(this);
     downloader.onDownloadStarted = this._onDownloadStarted.bind(this);
     downloader.onDownloadSuccess = this._onDownloadSuccess.bind(this);
     downloader.onDownloadError = this._onDownloadError.bind(this);
   },
 
   /**
    * Checks whether a subscription is currently being downloaded.
-   * @param {String} url  URL of the subscription
-   * @return {Boolean}
+   * @param {string} url  URL of the subscription
+   * @return {boolean}
    */
   isExecuting(url)
   {
     return downloader.isDownloading(url);
   },
 
   /**
    * Starts the download of a subscription.
-   * @param {DownloadableSubscription} subscription  Subscription to be
-   *                                                 downloaded
-   * @param {Boolean} manual  true for a manually started download
-   *                          (should not trigger fallback requests)
+   * @param {DownloadableSubscription} subscription
+   *   Subscription to be downloaded
+   * @param {boolean} manual
+   *   true for a manually started download (should not trigger fallback
+   *   requests)
    */
   execute(subscription, manual)
   {
     downloader.download(this._getDownloadable(subscription, manual));
   },
 
   /**
    * Yields Downloadable instances for all subscriptions that can be downloaded.
    */
   *_getDownloadables()
@@ skipping 205 matching lines @@
     {
       subscription.errors++;
 
       if (redirectCallback &&
           subscription.errors >= Prefs.subscriptions_fallbackerrors &&
           /^https?:\/\//i.test(subscription.url))
       {
         subscription.errors = 0;
 
         let fallbackURL = Prefs.subscriptions_fallbackurl;
-        let {addonVersion} = require("info");
+        const {addonVersion} = require("info");
         fallbackURL = fallbackURL.replace(/%VERSION%/g,
                                           encodeURIComponent(addonVersion));
         fallbackURL = fallbackURL.replace(/%SUBSCRIPTION%/g,
                                           encodeURIComponent(subscription.url));
         fallbackURL = fallbackURL.replace(/%URL%/g,
                                           encodeURIComponent(downloadURL));
         fallbackURL = fallbackURL.replace(/%ERROR%/g,
                                           encodeURIComponent(error));
         fallbackURL = fallbackURL.replace(/%CHANNELSTATUS%/g,
                                           encodeURIComponent(channelStatus));
         fallbackURL = fallbackURL.replace(/%RESPONSESTATUS%/g,
                                           encodeURIComponent(responseStatus));
 
         let request = new XMLHttpRequest();
         request.mozBackgroundRequest = true;
         request.open("GET", fallbackURL);
         request.overrideMimeType("text/plain");
         request.channel.loadFlags = request.channel.loadFlags |
                                     request.channel.INHIBIT_CACHING |
                                     request.channel.VALIDATE_ALWAYS;
         request.addEventListener("load", ev =>
         {
           if (onShutdown.done)
             return;
 
           if (!(subscription.url in FilterStorage.knownSubscriptions))
             return;
 
           let match = /^(\d+)(?:\s+(\S+))?$/.exec(request.responseText);
-          if (match && match[1] == "301" &&           // Moved permanently
+          if (match && match[1] == "301" &&    // Moved permanently
               match[2] && /^https?:\/\//i.test(match[2]))
+          {
             redirectCallback(match[2]);
-          else if (match && match[1] == "410")        // Gone
+          }
+          else if (match && match[1] == "410") // Gone
           {
             let data = "[Adblock]\n" +
-              subscription.filters.map((f) => f.text).join("\n");
+              subscription.filters.map(f => f.text).join("\n");
             redirectCallback("data:text/plain," + encodeURIComponent(data));
           }
         }, false);
         request.send(null);
       }
     }
   }
 };
 Synchronizer.init();