SDA WPTs batch3: remove reliance on previous sibling lookup in 3 tests
We have spec consensus on replacing previous-sibling lookup in
animation-timeline with deferred-timeline-attachment. This CL removes
previous sibling lookup from the following tests:
scroll-animations/css/scroll-timeline-in-container-query.html
scroll-animations/css/scroll-timeline-sampling.html
scroll-animations/css/scroll-timeline-sibling-gcs.html
Bug: 1432627
Change-Id: Ie3e7c8bbdca677e73938fc2fe39a422c9c250e62
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4470194
Reviewed-by: Kevin Ellis <kevers@chromium.org>
Commit-Queue: Mustaq Ahmed <mustaq@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1135925}
diff --git a/scroll-animations/css/scroll-timeline-in-container-query.html b/scroll-animations/css/scroll-timeline-in-container-query.html
index d6a8ba8..38b8ffd 100644
--- a/scroll-animations/css/scroll-timeline-in-container-query.html
+++ b/scroll-animations/css/scroll-timeline-in-container-query.html
@@ -21,6 +21,7 @@
width: auto;
height: 100px;
}
+
#scroller > div {
height: 200px;
}
@@ -32,14 +33,15 @@
}
}
- @keyframes expand {
+ @keyframes recolor {
from { background-color: rgb(100, 100, 100); }
to { background-color: rgb(200, 200, 200); }
}
+
#element {
height: 10px;
width: 10px;
- animation: expand 10s linear;
+ animation: recolor 10s linear;
animation-timeline: timeline;
background-color: rgb(0, 0, 0);
}
@@ -48,8 +50,7 @@
<div id=container>
<div id=scroller>
<div></div>
- </div>
- <div id=element>
+ <div id=element></div>
</div>
</div>
</div>
@@ -58,7 +59,7 @@
promise_test(async (t) => {
element.offsetTop;
- scroller.scrollTop = 50;
+ scroller.scrollTop = (scroller.scrollHeight - scroller.clientHeight) / 2;
await waitForNextFrame();
// Unknown timeline, time held at zero.
assert_equals(getComputedStyle(element).backgroundColor, 'rgb(100, 100, 100)');
diff --git a/scroll-animations/css/scroll-timeline-sampling.html b/scroll-animations/css/scroll-timeline-sampling.html
index 1fe354e..51b60e7 100644
--- a/scroll-animations/css/scroll-timeline-sampling.html
+++ b/scroll-animations/css/scroll-timeline-sampling.html
@@ -29,8 +29,8 @@
</style>
<div id=scroller>
<div id=contents></div>
+ <div id=element></div>
</div>
-<div id=element></div>
<script>
promise_test(async (t) => {
// The scroll timeline is initially inactive until the first frame.
