In ScrollableArea when there is an initial scroll-snap offset or while resnapping after layout, scroll offsets are interpreted as scroll positions. This leads to incorrect scroll positions when dealing with RTL layouts.
Created attachment 430462 [details]
Comment on attachment 430462 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=430462&action=review
Please check the test failures.
> +IntPoint ScrollableArea::nearestActiveSnapPoint(const IntPoint& currentOffset)
Maybe we should rename this nearestActiveSnapPointForOffset(). The argument should be a ScrollOffset.
It looks like these Mac failures are due to a related issue at bug 226630. I have a fix for that, but it depends on bug 226538.
Created attachment 431414 [details]
Committed r278868 (238812@main): <https://commits.webkit.org/238812@main>
All reviewed patches have been landed. Closing bug and clearing flags on attachment 431414 [details].