[WPT] Fix a flaky test resource_dedicated_worker.html.

Guohui Deng · chromium-wpt-export-bot · commit a015768bfde4 · 2025-11-06T18:01:08.000-08:00
It's flakey because the number of resource timing entries depends on whether `favicon.ico` request is included or not. Reference: web-platform-tests/interop#1078 (comment) A more reliable way to verify the resources from workers don't appear in the main document, is to verify the resources from workers don't appear in the main document. . Bug: 454485904 Change-Id: I375fa1191ee3ae594cb5302679ccd5195b9baaa2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7081867 Reviewed-by: Noam Rosenthal <nrosenthal@google.com> Commit-Queue: Guohui Deng <guohuideng@microsoft.com> Reviewed-by: Michal Mocny <mmocny@chromium.org> Cr-Commit-Position: refs/heads/main@{#1541263}
diff --git a/resource-timing/resource_dedicated_worker.html b/resource-timing/resource_dedicated_worker.html
@@ -9,16 +9,54 @@
 <script src="/resources/testharnessreport.js"></script>
 <script src="resources/webperftestharness.js"></script>
 <script src="resources/webperftestharnessextension.js"></script>
-<link rel="stylesheet" href="resources/resource_timing_test0.css" />
+<script src="/common/get-host-info.sub.js"></script>
 <script>
-setup({explicit_done: true});
+
+const mainDocResource = "/images/red.png?noCacheResourceDedicatedWorker";
+// Resources below are requested in the worker thread
+// "resources/worker_with_images.js".
+const workerResources = [
+  "/resource-timing/resources/blue.png",
+  "/resource-timing/resources/resource_timing_test0.png",
+];
+
+const {promise: mainDocPromise, resolve: resolveMainDoc} = Promise.withResolvers();
+const {promise: workerPromise, resolve: resolveWorker} = Promise.withResolvers();
+
+// Load main document resource
+var xhr = new XMLHttpRequest;
+xhr.open('get', mainDocResource, true);
+xhr.onload = function() {
+  resolveMainDoc();
+}
+xhr.send();
+
 const worker = new Worker("resources/worker_with_images.js");
 worker.onmessage = function(event) {
-  const context = new PerformanceContext(window.performance);
-  const entries = context.getEntriesByType('resource');
-  test_equals(entries.length, 6, "There should be six entries: 4 scripts, 1 stylesheet, and the worker itself");
-  done();
+  resolveWorker();
 }
+
+promise_test(async () => {
+  await Promise.all([
+    mainDocPromise,
+    workerPromise,
+  ]);
+
+  const context = new PerformanceContext(window.performance);
+
+  // The main doc resource appears in the main document.
+  const mainDocResourceUrl = new URL(mainDocResource, get_host_info().HTTP_ORIGIN).href;
+  const mainDocEntries = context.getEntriesByName(mainDocResourceUrl);
+  assert_greater_than(mainDocEntries.length, 0, "main doc resource appears");
+
+  // On the other hand, the worker resources don't.
+  for(resource of workerResources) {
+    let workerResourceUrl = new URL(resource, get_host_info().HTTP_ORIGIN).href;
+    let workerEntries = context.getEntriesByName(workerResourceUrl);
+    assert_equals(workerEntries.length, 0, "worker resources don't appear");
+  }
+}, "Verify that resources requested by dedicated workers don't appear in the main document");
+
 </script>
 </head>
 <body>
