Limit document.createEvent to existing events

Limit document.createEvent to existing events

Since https://codereview.chromium.org/1673243002 a number of event
interfaces have been added that are also supported in createEvent:

InputEvent: https://codereview.chromium.org/1682683002
ForeignFetchEvent: https://codereview.chromium.org/1845463002
PaymentRequestUpdateEvent: https://codereview.chromium.org/1931233002
SensorErrorEvent+SensorReadingEvent: https://codereview.chromium.org/1942663003

It doesn't make sense to continue expanding document.createEvent for
event interfaces which have no init*Event() methods, so limit support to
the existing set and hope to converge with spec and others over time.

This does not change the generated Event.cpp or EventModules.cpp.

BUG=569690
Committed: https://crrev.com/62d73936675d1e11d91dd6174a37856584345b47
Cr-Commit-Position: refs/heads/master@{#409788}

[foolip, 2016-08-04 09:57:34]

The CQ bit was checked by foolip@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-04 09:57:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[foolip, 2016-08-04 09:58:04]

Description was changed from

==========
Limit document.createEvent to existing events

Since https://codereview.chromium.org/1673243002 a number of event
interfaces have been added that are also supported in
document.createEvent:

InputEvent: https://codereview.chromium.org/1682683002
ForeignFetchEvent: https://codereview.chromium.org/1845463002
PaymentRequestUpdateEvent: https://codereview.chromium.org/1931233002
SensorErrorEvent+SensorReadingEvent: https://codereview.chromium.org/1942663003

It doesn't make sense to continue expanding document.createEvent for
event interfaces which have no init*Event() methods, so limit support to
the existing set and hope to converge with spec and others over time.

This does not change the generated Event.cpp or EventModules.cpp.

BUG=569690
==========

to

==========
Limit document.createEvent to existing events

Since https://codereview.chromium.org/1673243002 a number of event
interfaces have been added that are also supported in createEvent:

InputEvent: https://codereview.chromium.org/1682683002
ForeignFetchEvent: https://codereview.chromium.org/1845463002
PaymentRequestUpdateEvent: https://codereview.chromium.org/1931233002
SensorErrorEvent+SensorReadingEvent: https://codereview.chromium.org/1942663003

It doesn't make sense to continue expanding document.createEvent for
event interfaces which have no init*Event() methods, so limit support to
the existing set and hope to converge with spec and others over time.

This does not change the generated Event.cpp or EventModules.cpp.

BUG=569690
==========

[foolip, 2016-08-04 09:58:58]

foolip@chromium.org changed reviewers:
+ davve@opera.com, jochen@chromium.org

[foolip, 2016-08-04 09:58:59]

PTAL

[foolip, 2016-08-04 10:23:27]

See also https://codereview.chromium.org/2216593002

[commit-bot: I haz the power, 2016-08-04 11:26:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-04 11:26:31]

Dry run: This issue passed the CQ dry run.

[Mike West, 2016-08-04 15:15:20]

mkwst@chromium.org changed reviewers:
+ mkwst@chromium.org

[Mike West, 2016-08-04 15:15:21]

LGTM.

Did you consider sending an FYI to blink-dev@? (Or did you already?)

[foolip, 2016-08-04 15:18:46]

The CQ bit was checked by foolip@chromium.org

[commit-bot: I haz the power, 2016-08-04 15:19:16]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-04 15:23:06]

Description was changed from

==========
Limit document.createEvent to existing events

Since https://codereview.chromium.org/1673243002 a number of event
interfaces have been added that are also supported in createEvent:

InputEvent: https://codereview.chromium.org/1682683002
ForeignFetchEvent: https://codereview.chromium.org/1845463002
PaymentRequestUpdateEvent: https://codereview.chromium.org/1931233002
SensorErrorEvent+SensorReadingEvent: https://codereview.chromium.org/1942663003

It doesn't make sense to continue expanding document.createEvent for
event interfaces which have no init*Event() methods, so limit support to
the existing set and hope to converge with spec and others over time.

This does not change the generated Event.cpp or EventModules.cpp.

BUG=569690
==========

to

==========
Limit document.createEvent to existing events

Since https://codereview.chromium.org/1673243002 a number of event
interfaces have been added that are also supported in createEvent:

InputEvent: https://codereview.chromium.org/1682683002
ForeignFetchEvent: https://codereview.chromium.org/1845463002
PaymentRequestUpdateEvent: https://codereview.chromium.org/1931233002
SensorErrorEvent+SensorReadingEvent: https://codereview.chromium.org/1942663003

It doesn't make sense to continue expanding document.createEvent for
event interfaces which have no init*Event() methods, so limit support to
the existing set and hope to converge with spec and others over time.

This does not change the generated Event.cpp or EventModules.cpp.

BUG=569690
==========

[commit-bot: I haz the power, 2016-08-04 15:23:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-04 15:24:47]

Description was changed from

==========
Limit document.createEvent to existing events

Since https://codereview.chromium.org/1673243002 a number of event
interfaces have been added that are also supported in createEvent:

InputEvent: https://codereview.chromium.org/1682683002
ForeignFetchEvent: https://codereview.chromium.org/1845463002
PaymentRequestUpdateEvent: https://codereview.chromium.org/1931233002
SensorErrorEvent+SensorReadingEvent: https://codereview.chromium.org/1942663003

It doesn't make sense to continue expanding document.createEvent for
event interfaces which have no init*Event() methods, so limit support to
the existing set and hope to converge with spec and others over time.

This does not change the generated Event.cpp or EventModules.cpp.

BUG=569690
==========

to

==========
Limit document.createEvent to existing events

Since https://codereview.chromium.org/1673243002 a number of event
interfaces have been added that are also supported in createEvent:

InputEvent: https://codereview.chromium.org/1682683002
ForeignFetchEvent: https://codereview.chromium.org/1845463002
PaymentRequestUpdateEvent: https://codereview.chromium.org/1931233002
SensorErrorEvent+SensorReadingEvent: https://codereview.chromium.org/1942663003

It doesn't make sense to continue expanding document.createEvent for
event interfaces which have no init*Event() methods, so limit support to
the existing set and hope to converge with spec and others over time.

This does not change the generated Event.cpp or EventModules.cpp.

BUG=569690

Committed: https://crrev.com/62d73936675d1e11d91dd6174a37856584345b47
Cr-Commit-Position: refs/heads/master@{#409788}
==========

[commit-bot: I haz the power, 2016-08-04 15:24:48]

Patchset 1 (id:??) landed as
https://crrev.com/62d73936675d1e11d91dd6174a37856584345b47
Cr-Commit-Position: refs/heads/master@{#409788}

[foolip, 2016-08-04 15:35:39]

On 2016/08/04 15:15:21, Mike West wrote:
> LGTM.
> 
> Did you consider sending an FYI to blink-dev@? (Or did you already?)

Good idea!
https://groups.google.com/a/chromium.org/d/msg/blink-dev/bj1SUohzcR4/5SXyG3kb...