1655242 - relative scroll updates that come in during smooth scroll animations truncate the existing animation

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect, P3)

Description

[Timothy Nikkel (:tnikkel)]

Because we apply the relative offset to mScrollOffset and/or mSmoothScrollOffset we make the new destination the current position plus the relative offset. I know for pure relative offset updates we want the new destination to be the existing destination of the existing smooth scroll animation plus the pure relative offset. Less sure if we want the same for regular relative offset updates.

Comment 1

[Timothy Nikkel (:tnikkel)]

Attachment: Bug 1655242. When a relative scroll offset update comes in during a smooth scroll animation make the new destination be based on the existing destination, not the current scroll position. r?kats

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1e36ce18138c
When a relative scroll offset update comes in during a smooth scroll animation make the new destination be based on the existing destination, not the current scroll position. r=kats

Comment 3

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/1e36ce18138c