extension/*: subscribeToExtension: process all updates, use more spec…

…ific `isSafe` functions

This fixes some potential state display inconsistency bugs and improves UI
pages' init performance when the core is very busy.

Also, with this the logic inside `processUpdate` functions in the future patches
becomes way too complex.

extension/lib/lutils.js

@@ -78,12 +78,6 @@ function showSaved(suffix, ...args) {
     return showInternalPageAtNode("/page/saved.html" + suffix, ...args);
 }
 
-// a filter for core events that can be ignored without causing inconsistencies
-function thisIsFine(event) {
-    // it usually isn't, isn't it?
-    return false;
-}
-
 function setPageLoading() {
     document.getElementById("body_loading").innerHTML = "<p>Loading...</p>";
 }

extension/lib/utils.js

@@ -864,8 +864,6 @@ function subscribeToExtension(processUpdate, reinit, isSafe, markLoading, markSe
         }
         function processUpdateSmartly(event) {
             shouldReset = shouldReset || !isSafe(event);
-            if (shouldReset)
-                return;
             // apparently, this event can be processed synchronously
             processUpdateSync(event);
         }

extension/page/popup.js

@@ -405,7 +405,10 @@ async function popupMain() {
             await updateTabConfig();
             await updateTabStats();
         }
-    }), thisIsFine, setPageLoading, setPageSettling);
+    }), (event) => {
+        let cmd = event[0];
+        return !cmd.startsWith("update");
+    }, setPageLoading, setPageSettling);
 
     // show UI
     setPageLoaded();

extension/page/saved.js

@@ -67,7 +67,7 @@ async function stateMain() {
 
     await subscribeToExtension(catchAll(processUpdate), catchAll(async (willReset) => {
         await updateConfig();
-    }), thisIsFine, setPageLoading, setPageSettling);
+    }), () => true, setPageLoading, setPageSettling);
 
     let rrfilters = await browser.runtime.sendMessage(["getSavedFilters"]);
     await browser.runtime.sendMessage(["setSavedFilters", rrfilters]);

extension/page/state.js

@@ -159,7 +159,10 @@ async function stateMain() {
         resetLog(log);
         resetQueued(queuedLog);
         resetUnarchived(unarchivedLog);
-    }), thisIsFine, setPageLoading, setPageSettling);
+    }), (event) => {
+        let cmd = event[0];
+        return !(cmd.startsWith("reset") || cmd.startsWith("new"));
+    }, setPageLoading, setPageSettling);
 
     // show UI
     setPageLoaded();