1266154 - Momentum scrolling after scrolling a scroll frame with scroll snapping

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect)

Description

[Botond Ballo [:botond]]

I'm splitting out this bug from bug 1265513, which concerns the interaction between scroll snapping and handoff during panning (as opposed to after the finger is lifted).

There are two issues here, summarized in bug 1265513 comment 6.

The first one I have a fix for (originally posted in bug 1265513 comment 5).

The second one I argue is expected (in bug 1265513 comment 6), but I'm open to being convinced otherwise.

Comment 1

[Botond Ballo [:botond]]

Attachment: MozReview Request: Bug 1266154 - Don't start a scroll snap animation if we're already at the destination. r=kats

Review commit: https://reviewboard.mozilla.org/r/47739/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47739/

Comment 2

[Botond Ballo [:botond]]

Comment on attachment 8743401 [details]
MozReview Request: Bug 1266154 - Don't start a scroll snap animation if we're already at the destination. r=kats

Carrying r=kats from bug 1265513 comment 7.

Comment 3

[Botond Ballo [:botond]]

Try push that includes this patch: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f53dd840d47c

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e71e60f1459b

Comment 5

[Botond Ballo [:botond]]

Landed (with bug 1265515 as a ride-along).

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/e71e60f1459b