Allow all events to be accessed via on[foo] instead of just onFoo

Allow all events to be accessed via on[foo] instead of just onFoo

BUG=
R=blois@google.com

Committed: https://code.google.com/p/dart/source/detail?r=28966

[Emily Fortuna, 2013-10-19 00:57:31]

No tests, because these test things like Mouse Pointer Lock, and Speech and
fullscreen. :-P

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
File tools/dom/templates/html/impl/impl_EventTarget.darttemplate (right):

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:69: 'animationstart'
: 'onwebkitAnimationStart',
it seemed unrealistic to expect the user to know that "AnimationStart' starts
with a capital letter, but 'needkey' does not, so I made these case insensitive.
If you disagree, then we can just append "onwebkit" to the string and expect the
user to type it correctly.

[blois, 2013-10-19 01:19:09]

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
File tools/dom/templates/html/impl/impl_EventTarget.darttemplate (right):

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:58:
'fullscreenchange' : 'onwebkitfullscreenchange',
the on prefix should not be there.

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:62: 'keyadded' :
'onwebkitkeyadded',
Can we only add the webkit prefix on webkit browsers? Otherwise people could be
passing in valid event names then we screw it up.

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:66:
'resourcetimingbufferfull' : 'onwebkitresourcetimingbufferfull',
Looking at this list, there are more renames-
https://code.google.com/p/dart/source/browse/branches/bleeding_edge/dart/pkg/...

[Emily Fortuna, 2013-10-21 17:43:51]

ptal

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
File tools/dom/templates/html/impl/impl_EventTarget.darttemplate (right):

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:58:
'fullscreenchange' : 'onwebkitfullscreenchange',
On 2013/10/19 01:19:10, blois wrote:
> the on prefix should not be there.

Done.

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:62: 'keyadded' :
'onwebkitkeyadded',
On 2013/10/19 01:19:10, blois wrote:
> Can we only add the webkit prefix on webkit browsers? Otherwise people could
be
> passing in valid event names then we screw it up.

Done.

https://codereview.chromium.org/29843004/diff/1/tools/dom/templates/html/impl...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:66:
'resourcetimingbufferfull' : 'onwebkitresourcetimingbufferfull',
On 2013/10/19 01:19:10, blois wrote:
> Looking at this list, there are more renames-
>
https://code.google.com/p/dart/source/browse/branches/bleeding_edge/dart/pkg/...

I think those other events much have gone away. I can't find DOMFocusIn/Out in
html at all.

[blois, 2013-10-21 22:57:58]

https://codereview.chromium.org/29843004/diff/60001/tools/dom/templates/html/...
File tools/dom/templates/html/impl/impl_EventTarget.darttemplate (right):

https://codereview.chromium.org/29843004/diff/60001/tools/dom/templates/html/...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:77: if
(type.toLowerCase() in webkitEvents.keys) {
types are case-sensitive, toLowerCase should not be necessary.

https://codereview.chromium.org/29843004/diff/60001/tools/dom/templates/html/...
tools/dom/templates/html/impl/impl_EventTarget.darttemplate:78: var hasWebkit =
webkitEvents[type.toLowerCase()].startsWith('webkit');
are any of these not webkit?

[blois, 2013-10-21 23:02:28]

lgtm

[Emily Fortuna, 2013-10-21 23:07:25]

Committed patchset #3 manually as r28966 (presubmit successful).