Keep pseudo-elements during moveBefore

noamr · chromium-wpt-export-bot · commit d896bcbb5b1e · 2024-04-24T09:43:47.000-07:00
This happens by default since we don't detach the layout, but this CL removes the assertions that the psuedo-elements are gone. Bug: 40150299 Change-Id: I6200dd8ff0902b1e0d2950009ce46f51f53eda59 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5481991 Reviewed-by: Rune Lillesveen <futhark@chromium.org> Reviewed-by: Dominic Farolino <dom@chromium.org> Commit-Queue: Noam Rosenthal <nrosenthal@chromium.org> Cr-Commit-Position: refs/heads/main@{#1291905}
diff --git a/dom/nodes/moveBefore/tentative/continue-css-transition-left-pseudo.html b/dom/nodes/moveBefore/tentative/continue-css-transition-left-pseudo.html
@@ -0,0 +1,54 @@
+<!DOCTYPE html>
+<title>Node.moveBefore should preserve CSS transition state on pseudo-elements (left)</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+
+<body>
+  <section id="old-parent">
+    <div id="item"></div>
+  </section>
+  <section id="new-parent">
+  </section>
+  <style>
+    #item {
+      width: 100px;
+      height: 100px;
+      background: green;
+      position: absolute;
+      left: 0;
+    }
+
+    #item::before {
+      content: "Foo";
+      width: 100px;
+      height: 100px;
+      background: green;
+      transition: left 60s steps(1, jump-both);
+      left: 0px;
+      position: absolute;
+    }
+
+    #item.big::before {
+      left: 400px;
+    }
+
+    section {
+      position: relative;
+    }
+
+    body {
+      margin-left: 0;
+    }
+  </style>
+  <script>
+    promise_test(async t => {
+      const item = document.querySelector("#item");
+      assert_equals(item.getBoundingClientRect().x, 0);
+      item.classList.add("big");
+      await new Promise(resolve => item.addEventListener("transitionstart", resolve));
+      document.querySelector("#new-parent").moveBefore(item, null);
+      await new Promise(resolve => requestAnimationFrame(() => resolve()));
+      assert_equals(getComputedStyle(item, "::before").left, "200px");
+    });
+  </script>
+</body>
diff --git a/dom/nodes/moveBefore/tentative/continue-css-transition-transform-pseudo.html b/dom/nodes/moveBefore/tentative/continue-css-transition-transform-pseudo.html
@@ -0,0 +1,54 @@
+<!DOCTYPE html>
+<title>Node.moveBefore should preserve CSS transition state on pseudo-elements (transform)</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+
+<body>
+  <section id="old-parent">
+    <div id="item"></div>
+  </section>
+  <section id="new-parent">
+  </section>
+  <style>
+    #item {
+      width: 100px;
+      height: 100px;
+      background: green;
+      position: absolute;
+      left: 0;
+    }
+
+    #item::before {
+      content: "Foo";
+      width: 100px;
+      height: 100px;
+      background: green;
+      transition: transform 60s steps(1, jump-both);
+      transform: none;
+      position: absolute;
+    }
+
+    #item.big::before {
+      transform: translateX(400px);
+    }
+
+    section {
+      position: relative;
+    }
+
+    body {
+      margin-left: 0;
+    }
+  </style>
+  <script>
+    promise_test(async t => {
+      const item = document.querySelector("#item");
+      assert_equals(item.getBoundingClientRect().x, 0);
+      item.classList.add("big");
+      await new Promise(resolve => item.addEventListener("transitionstart", resolve));
+      document.querySelector("#new-parent").moveBefore(item, null);
+      await new Promise(resolve => requestAnimationFrame(() => resolve()));
+      assert_not_equals(getComputedStyle(item, "::before").transform, "none");
+    });
+  </script>
+</body>
