VT interop: Add waitForCompositorReady to several VT tests (#52701)

chromium-wpt-export-bot · vmpstr · web-flow · commit cb912e92b53e · 2025-05-23T18:54:50.000Z
This adds waiting for compositor ready to several tests. This ensures that the tests are resilient to start up conditions R=kevers@chromium.org Change-Id: I4babaf579d172454586f1d633ab4ce5c4cb139ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6569910 Reviewed-by: Kevin Ellis <kevers@chromium.org> Commit-Queue: Vladimir Levin <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/main@{#1463527} Co-authored-by: Vladimir Levin <vmpstr@chromium.org>
diff --git a/css/css-view-transitions/group-animation-for-root-transition.html b/css/css-view-transitions/group-animation-for-root-transition.html
@@ -4,12 +4,15 @@
 <link rel="help" href="https://drafts.csswg.org/css-view-transitions-1/">
 <link rel="author" href="mailto:khushalsagar@chromium.org">
 
+<script src="/dom/events/scrolling/scroll_support.js"></script>
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 
+<body>
 <script>
-promise_test(() => {
+promise_test(async () => {
   assert_implements(document.startViewTransition, "Missing document.startViewTransition");
+  await waitForCompositorReady();
   return new Promise(async (resolve, reject) => {
     let transition = document.startViewTransition();
     transition.ready.then(() => {
@@ -45,3 +48,4 @@
   });
 }, "incorrect UA animations for root transition");
 </script>
+</body>
diff --git a/css/css-view-transitions/no-crash-set-exception.html b/css/css-view-transitions/no-crash-set-exception.html
@@ -4,6 +4,7 @@
 <link rel="help" href="https://drafts.csswg.org/css-view-transitions-1/">
 <link rel="author" href="mailto:khushalsagar@chromium.org">
 
+<script src="/dom/events/scrolling/scroll_support.js"></script>
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 
@@ -30,6 +31,7 @@
 <script>
 promise_test(async t => {
   assert_implements(document.startViewTransition, "Missing document.startViewTransition");
+  await waitForCompositorReady();
   return new Promise((resolve, reject) => {
     document.startViewTransition(() => {
       resolve();
diff --git a/css/css-view-transitions/no-raf-while-render-blocked.html b/css/css-view-transitions/no-raf-while-render-blocked.html
@@ -5,6 +5,7 @@
 <link rel="help" href="https://drafts.csswg.org/css-view-transitions-1/">
 <link rel="author" href="mailto:khushalsagar@chromium.org">
 
+<script src="/dom/events/scrolling/scroll_support.js"></script>
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 
@@ -23,8 +24,9 @@
 <script>
 let renderBlocked = true;
 
-promise_test(() => {
+promise_test(async () => {
   assert_implements(document.startViewTransition, "Missing document.startViewTransition");
+  await waitForCompositorReady();
   return new Promise(async (resolve, reject) => {
     requestAnimationFrame(() => {
       document.startViewTransition(() => {
diff --git a/css/css-view-transitions/paused-animation-at-end.html b/css/css-view-transitions/paused-animation-at-end.html
@@ -3,6 +3,7 @@
 <link rel="help" href="https://www.w3.org/TR/css-view-transitions-1/">
 <link rel="author" href="mailto:vmpstr@chromium.org">
 
+<script src="/dom/events/scrolling/scroll_support.js"></script>
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 
@@ -25,6 +26,8 @@
 
 <script>
 promise_test(async (t) => {
+  assert_implements(document.startViewTransition, "Missing document.startViewTransition");
+  await waitForCompositorReady();
   return new Promise((resolve, reject) => {
     let transition = document.startViewTransition(() => {
       target.classList.replace("one", "two");
diff --git a/css/css-view-transitions/transition-skipped-after-animation-started.html b/css/css-view-transitions/transition-skipped-after-animation-started.html
@@ -4,6 +4,7 @@
 <link rel="help" href="https://www.w3.org/TR/css-view-transitions-1/">
 <link rel="author" href="mailto:khushalsagar@chromium.org">
 
+<script src="/dom/events/scrolling/scroll_support.js"></script>
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 
@@ -26,6 +27,7 @@
 <script>
 promise_test(async t => {
   assert_implements(document.startViewTransition, "Missing document.startViewTransition");
+  await waitForCompositorReady();
   return new Promise(async (resolve, reject) => {
     let transition = document.startViewTransition();
     await transition.ready;
diff --git a/css/css-view-transitions/view-transition-name-on-removed-element.html b/css/css-view-transitions/view-transition-name-on-removed-element.html
@@ -4,6 +4,7 @@
 <link rel="help" href="https://www.w3.org/TR/css-view-transitions-1/">
 <link rel="author" href="mailto:khushalsagar@chromium.org">
 
+<script src="/dom/events/scrolling/scroll_support.js"></script>
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 
@@ -23,6 +24,7 @@
 <script>
 promise_test(async t => {
   assert_implements(document.startViewTransition, "Missing document.startViewTransition");
+  await waitForCompositorReady();
   return new Promise(async (resolve, reject) => {
     // Remove an uncontained element. Because this element is not visited when
     // discovering named elements, the transition is not skipped.
