[$csp3 adblockpluschrome] Issue 5241 - Avoid setting badge text before navigation is committed

lib/stats.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 75 matching lines @@
         }
       }
 
       page.browserAction.setBadge(badge);
     }
   });
 });
 
 port.on("stats.getBlockedPerPage",
         message => getBlockedPerPage(new ext.Page(message.tab)));
+
+// We sometimes set the badge text before Chrome considers the tab to have been
+// updated. In those cases Chrome automatically clears the text again, so the
+// blocked count will only flash for a second. To work around that we wait for
+// the navigation to be committed and then update the text again.
+browser.webNavigation.onCommitted.addListener(details =>
+{
+  let page = new ext.Page({id: details.tabId});
+  let blocked = blockedPerPage.get(page);
+
+  if (blocked)
+  {
+    page.browserAction.setBadge({
+      color: badgeColor,
+      number: blocked
+    });
+  }
+});