Sync TouchEvent interfaces with the latest editors draft spec

Sync TouchEvent interfaces with the latest editors draft spec

BUG=None

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200320

[Rick Byers, 2015-08-10 16:39:06]

rbyers@chromium.org changed reviewers:
+ philipj@opera.com

[Rick Byers, 2015-08-10 16:39:06]

We've finished doing some cleanups / re-organization to the W3C TouchEvents
GitHub repo.  Seems like a good time to update the IDL comments.

[philipj_slow, 2015-08-11 08:33:23]

lgtm

https://codereview.chromium.org/1272603003/diff/1/Source/core/dom/Touch.idl
File Source/core/dom/Touch.idl (right):

https://codereview.chromium.org/1272603003/diff/1/Source/core/dom/Touch.idl#n...
Source/core/dom/Touch.idl:30: ] interface Touch {
TODO constructor? Looks like a few Touch members are missing from TouchInit in
the spec.

[Rick Byers, 2015-08-11 14:08:42]

On 2015/08/11 08:33:23, philipj wrote:
> lgtm
> 
> https://codereview.chromium.org/1272603003/diff/1/Source/core/dom/Touch.idl
> File Source/core/dom/Touch.idl (right):
> 
>
https://codereview.chromium.org/1272603003/diff/1/Source/core/dom/Touch.idl#n...
> Source/core/dom/Touch.idl:30: ] interface Touch {
> TODO constructor?

Added here and in TouchEvent.idl

> Looks like a few Touch members are missing from TouchInit in
> the spec.

Do you mean just pageX and pageY?  Those are computed from clientX and clientY,
see http://w3c.github.io/touch-events/#widl-TouchInit-clientX

[Rick Byers, 2015-08-11 14:09:05]

The CQ bit was checked by rbyers@chromium.org

[Rick Byers, 2015-08-11 14:09:06]

The patchset sent to the CQ was uploaded after l-g-t-m from philipj@opera.com
Link to the patchset: https://codereview.chromium.org/1272603003/#ps20001
(title: "Added todos for constructors")

[commit-bot: I haz the power, 2015-08-11 14:09:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1272603003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1272603003/20001

[philipj_slow, 2015-08-11 15:03:05]

On 2015/08/11 14:08:42, Rick Byers wrote:
> On 2015/08/11 08:33:23, philipj wrote:
> > Looks like a few Touch members are missing from TouchInit in
> > the spec.
> 
> Do you mean just pageX and pageY?  Those are computed from clientX and
clientY,
> see http://w3c.github.io/touch-events/#widl-TouchInit-clientX

I see. That's a bit unusual, most event constructors just allow all the values
to be set to anything at all, even if that's something that couldn't be produced
by a real event.

[Rick Byers, 2015-08-11 15:19:53]

On 2015/08/11 15:03:05, philipj wrote:
> On 2015/08/11 14:08:42, Rick Byers wrote:
> > On 2015/08/11 08:33:23, philipj wrote:
> > > Looks like a few Touch members are missing from TouchInit in
> > > the spec.
> > 
> > Do you mean just pageX and pageY?  Those are computed from clientX and
> clientY,
> > see http://w3c.github.io/touch-events/#widl-TouchInit-clientX
> 
> I see. That's a bit unusual, most event constructors just allow all the values
> to be set to anything at all, even if that's something that couldn't be
produced
> by a real event.

Let's continue this discussion in the right place:
https://github.com/w3c/touch-events/issues/28

[commit-bot: I haz the power, 2015-08-11 15:40:08]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200320