[Pointer Event] Enable pointerevent_sequence_at_implicit_release_on_drag-manual.html

RESOLVED FIXED in Firefox 58

Status

()

Core
DOM: Events
P2
normal
RESOLVED FIXED
11 months ago
4 months ago

People

(Reporter: stone, Assigned: stone)

Tracking

unspecified
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

Attachments

(3 attachments)

Comment hidden (empty)
(Assignee)

Updated

11 months ago
Assignee: nobody → sshih
(Assignee)

Comment 1

4 months ago
Created attachment 8906841 [details] [diff] [review]
Part1: Fire lostpointercapture event immediately after pointerup/pointercancel

Spec [1] says we should implicitly release pointer capture immediately after firing the pointerup or pointercancel events.

We may fire pointer leave / exit in EventStateManager::PostHandleEvent and implicit release pointer capture in current implementation.

Revise current implementation and handle it before PostHandleEvent.

[1] https://w3c.github.io/pointerevents/#implicit-release-of-pointer-capture
(Assignee)

Comment 2

4 months ago
Created attachment 8906842 [details] [diff] [review]
Part2: Only prevent drag when capturing the pointer events by content
(Assignee)

Comment 3

4 months ago
Created attachment 8906843 [details] [diff] [review]
Part3: Enable pointerevent_sequence_at_implicit_release_on_drag-manual.html
Priority: -- → P2
(Assignee)

Updated

4 months ago
Attachment #8906841 - Flags: review?(bugs)
(Assignee)

Updated

4 months ago
Attachment #8906842 - Flags: review?(bugs)
(Assignee)

Updated

4 months ago
Attachment #8906843 - Flags: review?(bugs)

Updated

4 months ago
Attachment #8906841 - Flags: review?(bugs) → review+

Updated

4 months ago
Attachment #8906843 - Flags: review?(bugs) → review+

Updated

4 months ago
Attachment #8906842 - Flags: review?(bugs) → review+

Comment 5

4 months ago
Pushed by sshih@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/aad29639fd5b
Part1: Fire lostpointercapture event immediately after pointerup/pointercancel. r=smaug.
https://hg.mozilla.org/integration/mozilla-inbound/rev/66ac49d8bfe7
Part2: Only prevent drag when capturing the pointer events by content. r=smaug.
https://hg.mozilla.org/integration/mozilla-inbound/rev/b2feeea8d65d
Part3: Enable pointerevent_sequence_at_implicit_release_on_drag-manual.html. r=smaug.

Comment 6

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/aad29639fd5b
https://hg.mozilla.org/mozilla-central/rev/66ac49d8bfe7
https://hg.mozilla.org/mozilla-central/rev/b2feeea8d65d
Status: NEW → RESOLVED
Last Resolved: 4 months ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.