From f4430cff32a4cb351825e5c88415d58ba4094400 Mon Sep 17 00:00:00 2001 From: Xianzhu Wang Date: Thu, 29 Aug 2024 17:45:30 -0700 Subject: [PATCH] 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 Commit-Queue: Xianzhu Wang Cr-Commit-Position: refs/heads/main@{#1348985} --- .../overflow-hidden-smooth-scroll-crash.html | 23 +++++++++++++++++++ 1 file changed, 23 insertions(+) create mode 100644 css/cssom-view/overflow-hidden-smooth-scroll-crash.html diff --git a/css/cssom-view/overflow-hidden-smooth-scroll-crash.html b/css/cssom-view/overflow-hidden-smooth-scroll-crash.html new file mode 100644 index 00000000000000..aed2a3dd2de968 --- /dev/null +++ b/css/cssom-view/overflow-hidden-smooth-scroll-crash.html @@ -0,0 +1,23 @@ + + + + + + A +
+ B +
C
+
+ + +