Bug 1915763 [wpt PR 47880] - Change where we force main-thread scrolling for overflow:hidden, a=testonly

wangxianzhu · moz-wptsync-bot · commit 45efcadc5dca · 2024-08-31T08:44:46.000Z
Automatic update from web-platform-tests Change where we force main-thread scrolling for overflow:hidden Previously the logic depended on RasterInducingScroll, and didn't work if the scroller is not painted. Now add main-thread scrolling on the blink scroll node in PaintPropertyTreeBuilder. Bug: 356436619 Change-Id: Ifebc2b33e6abdede7881d962407749df6742a1cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5823800 Reviewed-by: Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/main@{#1348985} -- wpt-commits: f4430cff32a4cb351825e5c88415d58ba4094400 wpt-pr: 47880
diff --git a/testing/web-platform/tests/css/cssom-view/overflow-hidden-smooth-scroll-crash.html b/testing/web-platform/tests/css/cssom-view/overflow-hidden-smooth-scroll-crash.html
@@ -0,0 +1,23 @@
+<!DOCTYPE html>
+<html class="test-wait">
+<link rel="help" href="https://drafts.csswg.org/cssom-view/#concept-smooth-scroll">
+<link rel="help" href="https://crbug.com/356436619">
+<body style="font-size: 4000px">
+  A
+  <div id=container style="overflow: hidden">
+    B
+    <div style="overflow: scroll">C</div>
+  </div>
+ </div>
+</body>
+<script>
+function doTest() {
+  container.scrollLeft = 1;
+  container.addEventListener('scrollend', () => {
+    document.documentElement.classList.remove('test-wait');
+  });
+  container.style.scrollBehavior = 'smooth';
+  container.scrollLeft = 20;
+}
+window.addEventListener('load', doTest);
+</script>
