Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 08e8584f2a997d928ced65716a03e9e4bd19a320
https://github.com/WebKit/WebKit/commit/08e8584f2a997d928ced65716a03e9e4bd19a320
Author: Simon Fraser <[email protected]>
Date: 2026-02-12 (Thu, 12 Feb 2026)
Changed paths:
A
LayoutTests/fast/scrolling/scroll-anchoring-in-overflow-scroll-expected.txt
A LayoutTests/fast/scrolling/scroll-anchoring-in-overflow-scroll.html
M Source/WebCore/page/scrolling/ScrollAnchoringController.cpp
M
Source/WebKit/UIProcess/RemoteLayerTree/ios/ScrollingTreeScrollingNodeDelegateIOS.mm
Log Message:
-----------
[iOS] [Scroll anchoring] Scroll anchoring in overflow scroll double-adjusts
https://bugs.webkit.org/show_bug.cgi?id=307600
rdar://170178709
Reviewed by Abrar Rahman Protyasha.
Scroll anchoring inside overflow:scroll on iOS was janky, because it
double-applied
the scroll delta. This happened because both
`ScrollingTreeScrollingNodeDelegateIOS`
and `ScrollingTreeScrollingNode` called `handleScrollPositionRequest()`.
Fix by removing the call in `ScrollingTreeScrollingNodeDelegateIOS`. I verified
that `setNeedsApplyLayerPositionsAfterCommit()` is always called already.
Test: fast/scrolling/scroll-anchoring-in-overflow-scroll.html
* LayoutTests/fast/scrolling/scroll-anchoring-in-overflow-scroll-expected.txt:
Added.
* LayoutTests/fast/scrolling/scroll-anchoring-in-overflow-scroll.html: Added.
* Source/WebCore/page/scrolling/ScrollAnchoringController.cpp:
(WebCore::ScrollAnchoringController::scrollPositionDidChange): Early return to
make logging clearer.
*
Source/WebKit/UIProcess/RemoteLayerTree/ios/ScrollingTreeScrollingNodeDelegateIOS.mm:
(WebKit::ScrollingTreeScrollingNodeDelegateIOS::commitStateAfterChildren):
Canonical link: https://commits.webkit.org/307434@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
