1682754 - Use setPointerCapture in Draggable.js

Status: RESOLVED FIXED

(DevTools :: Shared Components, task)

Description

[Nicolas Chevobbe [:nchevobbe]]

In draggable, we set a mousemove event on the document once the drag started, in order to keep track of mouse movements, even those "outside" of the drag element.
There's now a dedicated API for that, setPointerCapture, so let's use it instead.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1682754 - [devtools] Use setPointerCapture in Draggable.js . r=rcaliman.

We used to set the mousemove event listener on the document once the drag
started, so we could track mouse movement outside of the drag element.
This can now be done another way, using setPointerCapture, so we can have the
event listener directly on the element, and not on the document anymore.

A few tests that were dispatching mousemove events from the document are updated.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cdc175887602
[devtools] Use setPointerCapture in Draggable.js . r=rcaliman.

Comment 3

[Cristina Coroiu [:ccoroiu]]

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