Small fixes

- Adapt check groups for Chrome
- Add Error Boundary on other front files
- Force enable on lomanager tests

Author: Morikko

extension/background/core/groupmanager.js

@@ -426,7 +426,7 @@ GroupManager.updateLastAccessedFromTabs = function(groups = GroupManager.groups)
   for (let g of groups) {
     let lastAccessed = g.lastAccessed;
     for (let tab of g.tabs) { // If no tab, keep the old value
-      if (tab.lastAccessed > lastAccessed) {
+      if (tab.lastAccessed != null && tab.lastAccessed > lastAccessed) {
         lastAccessed = tab.lastAccessed;
       }
     }
@@ -1028,10 +1028,13 @@ GroupManager.checkCorruptedGroups = function(groups=GroupManager.groups, {
   withMessage=false,
 }={}) {
   const checkCorruptedTab = function(tab, index) {
-    const properties = [ "title", "url", "hidden", "lastAccessed", "pinned", "windowId", "active", "id"];
+    const properties = [ "title", "url", "pinned", "windowId", "active", "id"];
     if( Utils.isChrome() || Utils.isFF57()) {
       properties.push("discarded")
     }
+    if (!Utils.isChrome()) {
+      properties.push("lastAccessed")
+    }
     return Utils.ojectPropertiesAreUndefined(
       tab,
       properties,
@@ -1046,7 +1049,7 @@ GroupManager.checkCorruptedGroups = function(groups=GroupManager.groups, {
     if (results.length === 0) {
       return [false, ""];
     } else {
-      return [true, prefix + "." + results.map(([is, msg])=> msg).join(',')];
+      return [true, results.map(([is, msg])=>  `${prefix}.${msg}`).join(',')];
     }
   }
 

extension/tabpages/manage-groups/manage-groups.html

@@ -21,6 +21,8 @@
   <script src="/background/error/errorListenerFront.js"></script>
   <script src="/background/utils/utils.js"></script>
 
+  <script src="/share/components/ErrorBoundary.js"></script>
+
   <script src="/share/components/groups/wrapper/action_creators.js"></script>
   <script src="/share/components/groups/wrapper/reducer.js"></script>
   <script src="/share/components/groups/wrapper/group-actions.js"></script>

extension/tabpages/selector-groups/selector-groups.html

@@ -41,6 +41,8 @@
   <script src="/share/components/groups/wrapper/navigation.js"></script>
   -->
 
+  <script src="/share/components/ErrorBoundary.js"></script>
+
   <script src="/share/components/groups/searchbar.js"></script>
   <script src="/share/components/groups/grouplist.js"></script>
   <script src="/share/components/groups/group.js"></script>

extension/tests/tests/unit/test_logmanager.js

@@ -13,6 +13,7 @@ describe("Logmanager", () => {
                 logs,
                 print: false,
                 showNotification: false,
+                enable: true,
             })
 
             expect(logs[0].type).toBe('Error')
@@ -27,6 +28,7 @@ describe("Logmanager", () => {
                 logs,
                 print: false,
                 showNotification: false,
+                enable: true,
             })
 
             expect(logs.length).toBe(1);
@@ -44,6 +46,7 @@ describe("Logmanager", () => {
                 logs,
                 print: false,
                 showNotification: false,
+                enable: true,
             })
 
             expect(logs.length).toBe(1);