Issue 5459 - Add support to show a notification based on the number of ads blocked

lib/notification.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2017 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 199 matching lines @@
     let notifications = localData.concat(remoteData);
     if (notifications.length === 0)
       return null;
 
     const {addonName, addonVersion, application,
            applicationVersion, platform, platformVersion} = require("info");
 
     let targetChecks = {
       extension: v => v == addonName,
       extensionMinVersion:
-       v => Services.vc.compare(addonVersion, v) >= 0,

[Wladimir Palant, 2017/08/23 11:00:10]

Here and below: we shouldn't have one-space indentations, these aren't obvious
enough. It should always be at least two spaces.

[wspee, 2017/08/24 16:39:22]

Acknowledged.

+        v => Services.vc.compare(addonVersion, v) >= 0,
       extensionMaxVersion:
-       v => Services.vc.compare(addonVersion, v) <= 0,
+        v => Services.vc.compare(addonVersion, v) <= 0,
       application: v => v == application,
       applicationMinVersion:
-       v => Services.vc.compare(applicationVersion, v) >= 0,
+        v => Services.vc.compare(applicationVersion, v) >= 0,
       applicationMaxVersion:
-       v => Services.vc.compare(applicationVersion, v) <= 0,
+        v => Services.vc.compare(applicationVersion, v) <= 0,
       platform: v => v == platform,
       platformMinVersion:
-       v => Services.vc.compare(platformVersion, v) >= 0,
+        v => Services.vc.compare(platformVersion, v) >= 0,
       platformMaxVersion:
-       v => Services.vc.compare(platformVersion, v) <= 0,
-      blockedTotalMin: v => Prefs.blocked_total >= v,
-      blockedTotalMax: v => Prefs.blocked_total <= v,
-      locales: v => v.indexOf(Utils.appLocale) != -1

[Wladimir Palant, 2017/08/23 11:00:10]

Sorry, overlooked a detail in the previous review. This should be
v.includes(Utils.appLocale)

[wspee, 2017/08/24 16:39:22]

Acknowledged.

+        v => Services.vc.compare(platformVersion, v) <= 0,
+      blockedTotalMin: v => Prefs.show_statsinpopup &&
+        Prefs.blocked_total >= v,
+      blockedTotalMax: v => Prefs.show_statsinpopup &&
+        Prefs.blocked_total <= v,
+      locales: v => v.includes(Utils.appLocale)
     };
 
     let notificationToShow = null;
     for (let notification of notifications)
     {
       if (typeof notification.type === "undefined" ||
           notification.type !== "critical")
       {
         let shown;
         if (typeof Prefs.notificationdata.shown == "object")
@@ skipping 50 matching lines @@
         else
           continue;
       }
 
       if (notification.targets instanceof Array)
       {
         let match = false;
 
         for (let target of notification.targets)
         {
-          match = false;

[Wladimir Palant, 2017/08/23 11:00:11]

This assignment is no longer necessary.

[wspee, 2017/08/24 16:39:22]

Acknowledged.

           if (Object.keys(target).every(key =>
               targetChecks.hasOwnProperty(key) &&
               targetChecks[key](target[key])))
           {
             match = true;
             break;
           }
         }
         if (!match)
         {
@@ skipping 163 matching lines @@
     else if (index != -1 && forceValue !== true)
       categories.splice(index, 1);
 
     // HACK: JSON values aren't saved unless they are assigned a
     // different object.
     Prefs.notifications_ignoredcategories =
       JSON.parse(JSON.stringify(categories));
   }
 };
 Notification.init();