From 03c73786868d0a97c31b7b96282111105b2a1049 Mon Sep 17 00:00:00 2001
From: Karl Seguin <k@openmymind.io>
Date: Wed, 12 Aug 2026 19:12:37 +0800
Subject: [PATCH] Make report.log() a getter that can derive its value
 just-in-time

This actually showed up on a profile trace (was looking at something not
related). Rather than constantly calculating this, we only need to calculate
the value when it's called (both --dump wpt and wptrunner call this once after
report.complete is true).
---
 resources/testharnessreport.js | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/resources/testharnessreport.js b/resources/testharnessreport.js
index f1a34afb8c..4eb183d4c4 100644
--- a/resources/testharnessreport.js
+++ b/resources/testharnessreport.js
@@ -5,8 +5,19 @@
 var report = {
   complete: false,
   status: "",
-  log: "no test suite completion|Fail|The test never reaches the completion callback.",
   cases: {},
+
+  get log() {
+    const keys = Object.keys(report.cases);
+    if (keys.length === 0) {
+      return "no test suite completion|Fail|The test never reaches the completion callback.";
+    }
+    var log = "";
+    for (const k of keys) {
+      log += report.cases[k] + "\n";
+    }
+    return log;
+  },
   name: function(test) {
     const name = test.name;
     return name ? name.replace(/\n/g, '') : name;
@@ -17,24 +28,15 @@ var report = {
       log +=  "|"+test.message.replaceAll("\n"," ");
     }
     return log;
-  },
-  update: function() {
-    var log = "";
-    Object.keys(report.cases).forEach((k, i) => {
-      log += report.cases[k] + "\n";
-    });
-    report.log = log;
   }
 }
 
 add_test_state_callback(function (test) {
   report.cases[report.name(test)] = report.format(test);
-  report.update();
 });
 
 add_result_callback(function (test) {
   report.cases[report.name(test)] = report.format(test);
-  report.update();
 });
 
 add_completion_callback(function (tests, status) {