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}
web-platform-tests · Aug 30, 2024 · f4430cf · f4430cf
1 parent a97e8f7
commit f4430cf
Showing 1 changed file with 23 additions and 0 deletions.
diff --git a/css/cssom-view/overflow-hidden-smooth-scroll-crash.html b/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>