Bug 1841467 [wpt PR 40852] - Prerender: Re-enable prerender/service-workers.https.html, a=testonly

nhiroki · moz-wptsync-bot · commit b5be2e22dd3f · 2023-07-12T07:54:15.000Z
Automatic update from web-platform-tests Prerender: Re-enable prerender/service-workers.https.html `exec()` function returned by `create_prerendered_page()` is supposed to take an array object as the second param. Before this CL, it took a non-array object and fails several test cases in service-workers.https.html. This CL fixes them. Also, this CL updates the test expectation (.ini) based on an output of blink_tool.py. Change-Id: I06f488f566b306ac42db0ff0bb0800dec4be32f0 Bug: 1302831 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4659529 Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org> Reviewed-by: Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/main@{#1165439} -- wpt-commits: 8a85ac4b6f00ef970faa44518bb21b8aa6915438 wpt-pr: 40852
diff --git a/testing/web-platform/tests/speculation-rules/prerender/service-workers.https.html b/testing/web-platform/tests/speculation-rules/prerender/service-workers.https.html
@@ -1,5 +1,5 @@
 <!DOCTYPE html>
-<title>Same-origin prerendering can access Indexed Database</title>
+<title>Service Workers APIs with prerendering</title>
 <meta name="timeout" content="long">
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
@@ -44,7 +44,7 @@
           resolve({prerendering: document.prerendering});
         });
     });
-  }, scope);
+  }, [scope]);
 
   await new Promise(resolve => t.step_timeout(resolve, ACTIVATION_DELAY));
 
@@ -68,7 +68,7 @@
       await reg.unregister();
       return {prerendering: document.prerendering};
     })();
-  }, uid);
+  }, [uid]);
 
   await new Promise(resolve => t.step_timeout(resolve, ACTIVATION_DELAY));
 
@@ -92,7 +92,7 @@
       await reg.update();
       return {prerendering: document.prerendering};
     })();
-  }, uid);
+  }, [uid]);
 
   await new Promise(resolve => t.step_timeout(resolve, ACTIVATION_DELAY));
 
