From f72a354066175adf28c77865f19800b0336852bd Mon Sep 17 00:00:00 2001
From: egrs <egrs@users.noreply.github.com>
Date: Wed, 18 Feb 2026 07:15:17 +0100
Subject: [PATCH] address review: clear marks before test, assert exactly 2

---
 .../performance_observer/performance_observer.html | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/browser/tests/performance_observer/performance_observer.html b/src/browser/tests/performance_observer/performance_observer.html
index efa90e80..a0797e15 100644
--- a/src/browser/tests/performance_observer/performance_observer.html
+++ b/src/browser/tests/performance_observer/performance_observer.html
@@ -72,6 +72,9 @@
 
 <script id="buffered_option">
 {
+  // Clear marks from previous tests so we get a precise count
+  performance.clearMarks();
+
   // Create marks BEFORE the observer
   performance.mark("early1", { startTime: 1.0 });
   performance.mark("early2", { startTime: 2.0 });
@@ -81,17 +84,14 @@
     receivedEntries = list.getEntries();
   });
 
-  // With buffered: true, existing marks should be delivered immediately
+  // With buffered: true, existing marks should be delivered
   observer.observe({ type: "mark", buffered: true });
 
   testing.eventually(() => {
     testing.expectEqual(true, receivedEntries !== null);
-    // Should have received at least the 2 early marks
-    testing.expectEqual(true, receivedEntries.length >= 2);
-
-    const names = receivedEntries.map(e => e.name);
-    testing.expectEqual(true, names.indexOf("early1") >= 0);
-    testing.expectEqual(true, names.indexOf("early2") >= 0);
+    testing.expectEqual(2, receivedEntries.length);
+    testing.expectEqual("early1", receivedEntries[0].name);
+    testing.expectEqual("early2", receivedEntries[1].name);
 
     observer.disconnect();
   });