Intent to Implement and ship : Implement DragEvent and move MouseEvent.dataTransfer to DragEvent

53 views
Skip to first unread message

jinch...@navercorp.com

unread,
Jul 30, 2015, 6:11:30 AM7/30/15
to blink-dev

Contact emails

jinch...@navercorp.com


Spec

https://html.spec.whatwg.org/multipage/interaction.html#dragevent


Summary

Implement DragEvent and move MouseEvent.dataTransfer to DragEvent.


Motivation

Gecko alreay has the DragEvent interface and has the dataTransfer attribute only on that interface.


Compatibility risk

Gecko is already doing this. So it is unlikely to be problematic.


Ongoing technical constraints

None.


Will this feature be supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?

Yes.


OWP launch tracking bug?

http://crbug.com/498504


Requesting approval to ship?

Yes

Rick Byers

unread,
Jul 30, 2015, 2:38:35 PM7/30/15
to jinch...@navercorp.com, blink-dev
I support this.  It's a nice spec conformance and code clean-up (removes useless 'dataTransfer' attribute from MouseEvent) and brings us into line with Mozilla.  The risk seems very low, but like for other event properties we can't really tell from use counter data.

Philip Jägenstedt

unread,
Jul 31, 2015, 9:50:11 AM7/31/15
to Rick Byers, Jincheol Jo, blink-dev
LGTM

There is a bit of a spec issue here, around both DragEvent and ClipboardEvent:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=28984
https://github.com/w3c/clipboard-apis/issues/10

If anyone has input please comment on the GitHub issue, but either way
I trust it won't get in the way. A somewhat likely outcome is that the
DragEvent constructor won't change at all.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to blink-dev+...@chromium.org.

TAMURA, Kent

unread,
Aug 2, 2015, 11:08:19 PM8/2/15
to Philip Jägenstedt, Rick Byers, Jincheol Jo, blink-dev
LGTM2

--
TAMURA Kent
Software Engineer, Google


Chris Harrelson

unread,
Aug 3, 2015, 1:48:09 PM8/3/15
to TAMURA, Kent, Philip Jägenstedt, Rick Byers, Jincheol Jo, blink-dev
LGTM3
Reply all
Reply to author
Forward
0 new messages