Have ScrollAnimation work in terms of offsets, not positions
Created attachment 438569 [details] Patch
Created attachment 438577 [details] Patch
Created attachment 438598 [details] Patch
Comment on attachment 438598 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=438598&action=review > Source/WebCore/ChangeLog:8 > + Scroll positions can have negagive values in RTL content. It's simpler for ScrollAnimation negagive
Comment on attachment 438598 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=438598&action=review > Source/WebCore/platform/ScrollAnimationSmooth.cpp:76 > -bool ScrollAnimationSmooth::startAnimatedScrollToDestination(const FloatPoint& fromPosition, const FloatPoint& destinationPosition) > +bool ScrollAnimationSmooth::startAnimatedScrollToDestination(const FloatPoint& fromOffset, const FloatPoint& destinationOffset) I think the difference between "position" and "offset" here is too subtle and just changing names doesn't really make things clear. It would be nice to capture it to a type.
https://trac.webkit.org/changeset/282741/webkit
<rdar://problem/83289902>