Bug 158883 – TouchEvent should have a constructor

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Chris Dumez]

TouchEvent should have a constructor:
https://w3c.github.io/touch-events/#touchevent-interface

[Chris Dumez]

Created attachment 281577 [details]
Patch

[Benjamin Poulain]

Comment on attachment 281577 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=281577&action=review

> LayoutTests/fast/events/touch/touch-event-constructor.html:8
> +description("Tests the TouchEvent constructor");

Tests->Test

> LayoutTests/fast/events/touch/touch-event-constructor.html:30
> +shouldNotThrow("ev = new TouchEvent('touchmove', { touches: listA, targetTouches: listB, changedTouches: listC, bubbles: true })");
> +shouldBe("ev.__proto__", "TouchEvent.prototype");
> +shouldBeEqualToString("ev.type", "touchmove");
> +shouldBe("ev.touches.length", "1");
> +shouldBe("ev.targetTouches.length", "2");
> +shouldBe("ev.changedTouches.length", "3");
> +shouldBeTrue("ev.bubbles");

Ideally you should also cover each list separately.

[Chris Dumez]

Created attachment 281585 [details]
Patch

[Chris Dumez]

Comment on attachment 281585 [details]
Patch

Clearing flags on attachment: 281585

Committed r202178: <http://trac.webkit.org/changeset/202178>

[Chris Dumez]

All reviewed patches have been landed.  Closing bug.