Issue 7257 - Throttle badge updates

ext/background.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present 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 507 matching lines @@
             return frame.parent || null;
           return frames.get(0) || null;
         }
       };
     }
 
     return ext.onMessage._dispatch(
       message, sender, sendResponse
     ).includes(true);
   });
+
+
+  /* Utilities */
+
+  let Scheduler = ext.Scheduler = function()
+  {
+    this.scheduled = false;
+  };
+  Scheduler.prototype = {
+    schedule(delay, func)
+    {
+      setTimeout(() =>
+      {
+        this.scheduled = false;
+        func();
+      },
+      delay);
+
+      this.scheduled = true;
+    }
+  };

[Sebastian Noack, 2019/02/03 23:18:03]

Please don't add new stuff to ext.*, we eventually want to get rid of it.

[Manish Jethani, 2019/02/04 05:33:03]

I've moved it into lib/stats.js for now, although I think (1) it should be in
its own file, because I see this pattern in lib/indexedDBBackup.js and
lib/requestBlocker.js, and (2) it should be an ES2015 class.

[Sebastian Noack, 2019/02/04 06:32:04]

Well, IMO this pattern isn't worth to be abstracted in a class in the first
place. I'd ust use setTimeout() directly as we do in those other places.

[Manish Jethani, 2019/02/04 06:57:17]

Acknowledged.

 }