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
  * 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/>.
  */
 
 "use strict";
 
 let {createSandbox} = require("./_common");
 
 let Filter = null;
 let Subscription = null;
 let SpecialSubscription = null;
 let DownloadableSubscription = null;
 let RegularSubscription = null;
 let ExternalSubscription = null;
+let FilterNotifier = null;
 
 exports.setUp = function(callback)
 {
   let sandboxedRequire = createSandbox();
   ({Filter} = sandboxedRequire("../lib/filterClasses"));
   (
     {
       Subscription, SpecialSubscription, DownloadableSubscription
     } = sandboxedRequire("../lib/subscriptionClasses")
   );
+  ({FilterNotifier} = sandboxedRequire("../lib/filterNotifier"));
   callback();
 };
 
 function compareSubscription(test, url, expected, postInit)
 {
   expected.push("[Subscription]")
   let subscription = Subscription.fromURL(url);
   if (postInit)
     postInit(subscription)
   let result = subscription.serialize().trim().split("\n");
@@ skipping 203 matching lines @@
 
   subscription.removeFilterAt(0);
   test.equal(subscription.serializeFilters(), "[Subscription filters]\nfilter2\nfilter1\n", "One filter removed");
 
   subscription.delete();
   filter1.delete();
   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.

+    {
+      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);
+  }
+
+  let subscription = Subscription.fromURL("http://example.com/");
+  checkNotifications(() =>
+  {
+    subscription.title = "foobar";
+  }, ["subscription.title", "http://example.com/"], "Changing subscription title");
+  checkNotifications(() =>
+  {
+    subscription.title = "foobar";
+  }, null, "Setting subscription title to same value");
+  checkNotifications(() =>
+  {
+    subscription.title = null;
+  }, ["subscription.title", "http://example.com/"], "Resetting subscription title");
+
+  checkNotifications(() =>
+  {
+    subscription.disabled = true;
+  }, ["subscription.disabled", "http://example.com/"], "Disabling subscription");
+  checkNotifications(() =>
+  {
+    subscription.disabled = true;
+  }, null, "Disabling already disabled subscription");
+  checkNotifications(() =>
+  {
+    subscription.disabled = false;
+  }, ["subscription.disabled", "http://example.com/"], "Enabling subscription");
+
+  checkNotifications(() =>
+  {
+    subscription.fixedTitle = true;
+  }, ["subscription.fixedTitle", "http://example.com/"], "Marking title as fixed");
+  checkNotifications(() =>
+  {
+    subscription.fixedTitle = false;
+  }, ["subscription.fixedTitle", "http://example.com/"], "Marking title as editable");
+
+  checkNotifications(() =>
+  {
+    subscription.homepage = "http://example.info/";
+  }, ["subscription.homepage", "http://example.com/"], "Changing subscription homepage");
+  checkNotifications(() =>
+  {
+    subscription.homepage = null;
+  }, ["subscription.homepage", "http://example.com/"], "Resetting subscription homepage");
+
+  checkNotifications(() =>
+  {
+    subscription.lastCheck = 1234;
+  }, ["subscription.lastCheck", "http://example.com/"], "Changing subscription.lastCheck");
+  checkNotifications(() =>
+  {
+    subscription.lastCheck = 1234;
+  }, null, "Setting subscription.lastCheck to same value");
+  checkNotifications(() =>
+  {
+    subscription.lastCheck = 0;
+  }, ["subscription.lastCheck", "http://example.com/"], "Resetting subscription.lastCheck");
+
+  checkNotifications(() =>
+  {
+    subscription.lastDownload = 4321;
+  }, ["subscription.lastDownload", "http://example.com/"], "Changing subscription.lastDownload");
+  checkNotifications(() =>
+  {
+    subscription.lastDownload = 0;
+  }, ["subscription.lastDownload", "http://example.com/"], "Resetting subscription.lastDownload");
+
+  checkNotifications(() =>
+  {
+    subscription.downloadStatus = "ok";
+  }, ["subscription.downloadStatus", "http://example.com/"], "Changing subscription.downloadStatus");
+  checkNotifications(() =>
+  {
+    subscription.downloadStatus = null;
+  }, ["subscription.downloadStatus", "http://example.com/"], "Resetting subscription.downloadStatus");
+
+  checkNotifications(() =>
+  {
+    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();
+};