Closed Bug 1251915 Opened 4 years ago Closed 4 years ago

AccessibleCaret cannot be dragged when pointer events is enabled

Categories

(Core :: Selection, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

When "dom.w3c_pointer_events.enabled" is true, AccessibleCaretEventHub will receives an eTouchCancel immediately after eTouchStart, which will be treated as if the user lifts his finger from the touch screen. Thus the subsequent eTouchMove events never be processed to move the caret.

[1] https://dxr.mozilla.org/mozilla-central/rev/5e0140b6d11821e0c2a2de25bc5431783f03380a/gfx/layers/apz/util/APZEventState.cpp#323,331
In my original design, I treat eTouchCancel to be like eTouchEnd for
ending a caret dragging procedure.

However when pointer events is enabled, it sents an eTouchCancel event
after the eTouchStart event whose primary usage is to be converted to
pointer events, which then cancels the normal caret dragging procedure.

Moreover, when pointer events is disabled, we don't get eTouchCancel
during a normal caret dragging scenario, so we don't really need to
handle eTouchCancel anyway.

Review commit: https://reviewboard.mozilla.org/r/37085/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/37085/
Attachment #8724538 - Flags: review?(roc)
This patch is generatedy by applying clang-format on
AccessibleCaretEventHub.cpp.

Review commit: https://reviewboard.mozilla.org/r/37091/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/37091/
Attachment #8724544 - Flags: review?(roc)
Comment on attachment 8724538 [details]
MozReview Request: Bug 1251915 - Ignore handling eTouchCancel events. r?roc

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/37085/diff/1-2/
Comment on attachment 8724544 [details]
MozReview Request: Bug 1251915 - Correct AccessibleCaretEventHub coding style by clang-format. r?roc

https://reviewboard.mozilla.org/r/37091/#review33613
Attachment #8724544 - Flags: review?(roc) → review+
Comment on attachment 8724538 [details]
MozReview Request: Bug 1251915 - Ignore handling eTouchCancel events. r?roc

https://reviewboard.mozilla.org/r/37085/#review33615

Test?
Attachment #8724538 - Flags: review?(roc) → review+
Comment on attachment 8724544 [details]
MozReview Request: Bug 1251915 - Correct AccessibleCaretEventHub coding style by clang-format. r?roc

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/37091/diff/1-2/
Comment on attachment 8724538 [details]
MozReview Request: Bug 1251915 - Ignore handling eTouchCancel events. r?roc

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/37085/diff/2-3/
https://hg.mozilla.org/mozilla-central/rev/51b5b9c96032
https://hg.mozilla.org/mozilla-central/rev/50cb73a32974
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.