Issue 2420 - Move notification show logic to core

lib/notification.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 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/>.
  */
 
 /**
  * @fileOverview Handles notifications.
  */
 
 Cu.import("resource://gre/modules/Services.jsm");
 
 let {Prefs} = require("prefs");
 let {Downloader, Downloadable, MILLIS_IN_MINUTE, MILLIS_IN_HOUR, MILLIS_IN_DAY} = require("downloader");
 let {Utils} = require("utils");
 let {Matcher} = require("matcher");
 let {Filter} = require("filterClasses");
 
 let INITIAL_DELAY = 12 * MILLIS_IN_MINUTE;

[Felix Dahlke, 2015/06/08 19:34:35]

Nevermind this, just based this commit on an earlier one and this is the only
change reverted by that.

 let CHECK_INTERVAL = 1 * MILLIS_IN_HOUR;
 let EXPIRATION_INTERVAL = 1 * MILLIS_IN_DAY;
 let STARTUP_SHOW_DELAY = 3 * MILLIS_IN_MINUTE;
 let TYPE = {
   information: 0,
   question: 1,
   critical: 2
 };
 
 let showListeners = [];
@@ skipping 136 matching lines @@
    * @param {Function} listener Listener that was added via addShowListener()
    */
   removeShowListener: function(listener)
   {
     let index = showListeners.indexOf(listener);
     if (index != -1)
       showListeners.splice(index, 1);
   },
 
   /**
-   * Removes all listeners added via addShowListener().
-   */
-  removeAllShowListeners: function()
-  {
-    showListeners = [];
-  },
-
-  /**
    * Determines which notification is to be shown next.
    * @param {String} url URL to match notifications to (optional)
    * @return {Object} notification to be shown, or null if there is none
    */
   _getNextToShow: function(url)
   {
     function checkTarget(target, parameter, name, version)
     {
       let minVersionKey = parameter + "MinVersion";
       let maxVersionKey = parameter + "MaxVersion";
@@ skipping 165 matching lines @@
 
   /**
    * Notifies question listeners about interactions with a notification
    * @param {String} id notification ID
    * @param {Boolean} approved indicator whether notification has been approved or not
    */
   triggerQuestionListeners: function(id, approved)
   {
     if (!(id in questionListeners))
       return;
-    let questionListenersForId = questionListeners[id];
-    for (let listener of questionListenersForId)
+    let listeners = questionListeners[id];
+    for (let listener of listeners)
       listener(approved);
   },
   
   /**
    * Toggles whether notifications of a specific category should be ignored
    * @param {String} category notification category identifier
    * @param {Boolean} [forceValue] force specified value
    */
   toggleIgnoreCategory: function(category, forceValue)
   {
     let categories = Prefs.notifications_ignoredcategories;
     let index = categories.indexOf(category);
     if (index == -1 && forceValue !== false)
     {
       categories.push(category);
       Prefs.notifications_showui = true;
     }
     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();