Issue 5063 - [emscripten] Make FilterNotifier calls more efficient

test/filterClasses.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 359 matching lines @@
   }
 
   test.done();
 };
 
 exports.testNotifications = function(test)
 {
   function checkNotifications(action, expected, message)
   {
     let result = null;
-    let listener = (topic, filter, newValue, oldValue) =>
+    let listener = (topic, filter) =>
     {
       if (result)
         test.ok(false, "Got more that one notification - " + message);
       else
-        result = [topic, filter.text, newValue, oldValue];
+        result = [topic, filter.text];
     };
     FilterNotifier.addListener(listener);
     action();
     FilterNotifier.removeListener(listener);
     test.deepEqual(result, expected, message);
   }
 
   let filter = Filter.fromText("foobar");
   checkNotifications(() =>
   {
     filter.disabled = true;
-  }, ["filter.disabled", "foobar", true, false], "Disabling filter");
+  }, ["filter.disabled", "foobar"], "Disabling filter");
   checkNotifications(() =>
   {
     filter.disabled = true;
   }, null, "Disabling already disabled filter");
   checkNotifications(() =>
   {
     filter.disabled = false;
-  }, ["filter.disabled", "foobar", false, true], "Enabling filter");
+  }, ["filter.disabled", "foobar"], "Enabling filter");
 
   checkNotifications(() =>
   {
     filter.lastHit = 1234;
-  }, ["filter.lastHit", "foobar", 1234, 0], "Changing last filter hit");
+  }, ["filter.lastHit", "foobar"], "Changing last filter hit");
   checkNotifications(() =>
   {
     filter.lastHit = 1234;
   }, null, "Changing last filter hit to same value");
   checkNotifications(() =>
   {
     filter.lastHit = 0;
-  }, ["filter.lastHit", "foobar", 0, 1234], "Resetting last filter hit");
+  }, ["filter.lastHit", "foobar"], "Resetting last filter hit");
 
   checkNotifications(() =>
   {
     filter.hitCount++;
-  }, ["filter.hitCount", "foobar", 1, 0], "Increasing filter hit counts");
+  }, ["filter.hitCount", "foobar"], "Increasing filter hit counts");
   checkNotifications(() =>
   {
     filter.hitCount = 0;
-  }, ["filter.hitCount", "foobar", 0, 1], "Resetting filter hit counts");
+  }, ["filter.hitCount", "foobar"], "Resetting filter hit counts");
 
   filter.delete();
   test.done();
 };