Issue 5063 - [emscripten] Make FilterNotifier calls more efficient

test/subscriptionClasses.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 248 matching lines @@
   filter2.delete();
 
   test.done();
 };
 
 exports.testNotifications = function(test)
 {
   function checkNotifications(action, expected, message)
   {
     let result = null;
-    let listener = (topic, subscription, newValue, oldValue) =>

[sergei, 2017/04/19 12:18:03]

newValue and oldValue arguments are not used anymore.

[Wladimir Palant, 2017/04/20 08:03:14]

Done.

+    let listener = (topic, subscription) =>
     {
       if (result)
         test.ok(false, "Got more that one notification - " + message);
       else
         result = [topic, subscription.url];
     };
     FilterNotifier.addListener(listener);
     action();
     FilterNotifier.removeListener(listener);
     test.deepEqual(result, expected, message);
@@ skipping 80 matching lines @@
     subscription.errors++;
   }, ["subscription.errors", "http://example.com/"], "Increasing subscription.errors");
   checkNotifications(() =>
   {
     subscription.errors = 0;
   }, ["subscription.errors", "http://example.com/"], "Resetting subscription.erros");
 
   subscription.delete();
   test.done();
 };