Add comment pointing to RemoteContext doc.

Add a comment that points to the RemoteContext README. Also removed execute() line as it is already called in constructor. Change-Id: Idac381041f4499c5fdb431ac4971ba1f2ad5ceb5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3975550 Commit-Queue: Hao Liu <haoliuk@chromium.org> Reviewed-by: Annie Sullivan <sullivan@chromium.org> Cr-Commit-Position: refs/heads/main@{#1063015}
web-platform-tests · Oct 24, 2022 · b0abf2e · b0abf2e
1 parent 28f7d6c
commit b0abf2e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/navigation-timing/prefetch-transfer-size-executor.html b/navigation-timing/prefetch-transfer-size-executor.html
@@ -8,7 +8,15 @@
   <script src="/common/utils.js"></script>
   <script src="/common/dispatcher/dispatcher.js"></script>
 </head>
-
+<!--
+  This test uses RemoteContext and Executor to execute JavaScript in another
+  page opened by this test page. This is because we need to simulate
+  navigating away from current page in this test. It will cause all Javascript
+  code on the test page to be lost if we do it in the test page. We can't do it
+  in an iframe because currently the prefetch behaves differently in an iframe
+  in this particular test case. For detailed information about RemoteContext and
+  Executor, see /common/dispatcher/README.md.
+-->
 <body>
   <script>
     const addLink = (url) => {

diff --git a/navigation-timing/resources/blank_page_prefetch.html b/navigation-timing/resources/blank_page_prefetch.html
@@ -20,7 +20,6 @@
     const params = new URLSearchParams(window.location.search);
     const uuid = params.get("uuid");
     const executor = new Executor(uuid);
-    executor.execute();
   </script>
 
 </body>