Add safe checks (#4379)

sitespeedio · Dec 30, 2024 · 3662ef1 · 3662ef1
1 parent d4550ae
commit 3662ef1
Showing 1 changed file with 16 additions and 10 deletions.
diff --git a/lib/plugins/browsertime/browsertimeAggregator.js b/lib/plugins/browsertime/browsertimeAggregator.js
@@ -234,16 +234,20 @@ export class BrowsertimeAggregator {
         summary.userTimings = {};
         const marksData = {},
           measuresData = {};
-        for (const [timingName, stats] of Object.entries(
-          object.userTimings.marks
-        )) {
-          setStatsSummary(marksData, timingName, stats);
+        if (object.userTimings.marks) {
+          for (const [timingName, stats] of Object.entries(
+            object.userTimings.marks
+          )) {
+            setStatsSummary(marksData, timingName, stats);
+          }
         }
 
-        for (const [timingName, stats] of Object.entries(
-          object.userTimings.measures
-        )) {
-          setStatsSummary(measuresData, timingName, stats);
+        if (object.userTimings.measures) {
+          for (const [timingName, stats] of Object.entries(
+            object.userTimings.measures
+          )) {
+            setStatsSummary(measuresData, timingName, stats);
+          }
         }
 
         summary.userTimings.marks = marksData;
@@ -253,8 +257,10 @@ export class BrowsertimeAggregator {
         const categories = {};
         summary.cpu = {};
 
-        for (const [name, stats] of Object.entries(object.cpu.longTasks)) {
-          setStatsSummary(longTasks, name, stats);
+        if (object.cpu.longTasks) {
+          for (const [name, stats] of Object.entries(object.cpu.longTasks)) {
+            setStatsSummary(longTasks, name, stats);
+          }
         }
 
         if (object.cpu.categories) {