Issue 3839 - Adapt for Prefs event changes

background.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 39 matching lines @@
     }
   };
 
   function Notifier()
   {
     this._eventEmitter = new EventEmitter();
   }
   Notifier.prototype = {
     addListener: function(listener)
     {
-      var listeners = this._eventEmitter[""];

[Thomas Greiner, 2016/03/22 13:29:40]

Detail: Shouldn't this be `var listeners = this._eventEmitter._listeners[""];`?

Functionality-wise there's no difference but it doesn't seem to be done on
purpose.

[Sebastian Noack, 2016/03/22 14:24:10]

Well spotted, and actually it is a bug.

+      var listeners = this._eventEmitter._listeners[""];
       if (!listeners || listener.indexOf(listener) == -1)
         this._eventEmitter.on("", listener);
     },
     removeListener: function(listener)
     {
       this._eventEmitter.off("", listener);
     },
     triggerListeners: function()
     {
       var args = Array.prototype.slice.apply(arguments);
@@ skipping 421 matching lines @@
 
   if (params.safariContentBlocker)
   {
     global.safari = {
       extension: {
         setContentBlocker: function() {}
       }
     };
   }
 })(this);