Closed
Bug 1344108
Opened 7 years ago
Closed 7 years ago
[Pointer Event] Enable pointerevent_sequence_at_implicit_release_on_drag-manual.html
Categories
(Core :: DOM: Events, enhancement, P2)
Core
DOM: Events
Tracking
()
RESOLVED
FIXED
mozilla58
Tracking | Status | |
---|---|---|
firefox58 | --- | fixed |
People
(Reporter: stone, Assigned: stone)
Details
Attachments
(3 files)
4.32 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
1.43 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
4.37 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → sshih
Assignee | ||
Comment 1•7 years ago
|
||
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•7 years ago
|
||
Assignee | ||
Comment 3•7 years ago
|
||
Updated•7 years ago
|
Priority: -- → P2
Assignee | ||
Updated•7 years ago
|
Attachment #8906841 -
Flags: review?(bugs)
Assignee | ||
Updated•7 years ago
|
Attachment #8906842 -
Flags: review?(bugs)
Assignee | ||
Updated•7 years ago
|
Attachment #8906843 -
Flags: review?(bugs)
Updated•7 years ago
|
Attachment #8906841 -
Flags: review?(bugs) → review+
Updated•7 years ago
|
Attachment #8906843 -
Flags: review?(bugs) → review+
Updated•7 years ago
|
Attachment #8906842 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 4•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=b5db3283b516bb7e28479132edaeb902c98b3eb5
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•7 years 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
Closed: 7 years 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.
Description
•