1246477 - AccessibleCaret is not updated if the input is being scrolled after long pressing

Status: RESOLVED FIXED

(Core :: DOM: Selection, defect)

Description

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

This is a regression by "Bug 1121468 - Go to NoActionState after receiving release on LongTapState."

Steps to reproduce on Fennec:
1. Open Google search, and type some text to the search field.
2. Single tap elsewhere to make the keyboard go away.
3. Long tap to select a word on the search field without lifting the finger. This is necessary to reproduce this issue.
4. Observe the search field will be scrolled into the center of the screen.

Expected result:
The position of the carets is being updated to the correct place.

Actual results:
The position of the carets is at the wrong place.

Comment 1

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

Attachment: MozReview Request: Bug 1246477 - Fix carets not updated by scroll events in LongTapState. r=roc

This is a regression by "Bug 1121468 - Go to NoActionState after
receiving release on LongTapState."

When receiving a scroll event in LongTapState, i.e. apz starts, we
should call OnScrollStart() and move to the ScrollState.

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

Comment 2

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 8716861 [details]
MozReview Request: Bug 1246477 - Fix carets not updated by scroll events in LongTapState. r=roc

https://reviewboard.mozilla.org/r/33963/#review30559

Comment 3

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=77c02afb6517&selectedJob=16473427

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/714c60cf29d9

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/714c60cf29d9