See https://github.com/whatwg/html/issues/3487
Spec pull request: https://github.com/whatwg/html/pull/4129 . It looks like we ended up using GlobalEventHandlers. We are also considering adding it to ShadowRoot, but there's no spec PR for that yet.
There is now and I suspect we'll merge it once an editorial matter is resolved: https://github.com/whatwg/dom/pull/785.
https://github.com/whatwg/html/pull/4129
Created attachment 410325 [details] Patch
Created attachment 410334 [details] Patch
Created attachment 410337 [details] Patch
Comment on attachment 410337 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410337&action=review Looks okay to me but your change logs need some work. > Source/WebCore/ChangeLog:7 > + Could you add a description here? > Source/WebCore/ChangeLog:9 > + Tests: > + imported/w3c/web-platform-tests/dom/idlharness.window.html: This isn't right. The first test name should appear after "Tests:" followed by a single space, and the subsequent lines should appear aligned to that line: e.g. Tests: imported/w3c/web-platform-tests/dom/idlharness.window.html imported/w3c/web-platform-tests/html/webappapis/scripting/events/event-handler-all-global-events.html ... > LayoutTests/ChangeLog:7 > + Say that you've rebaselined tests now that more test cases are passing. > LayoutTests/imported/w3c/ChangeLog:7 > + Ditto.
Comment on attachment 410337 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410337&action=review >> Source/WebCore/ChangeLog:7 >> + > > Could you add a description here? I thought this was self-obvious. I'll add a description.
Created attachment 410424 [details] Patch
Committed r267939: <https://trac.webkit.org/changeset/267939> All reviewed patches have been landed. Closing bug and clearing flags on attachment 410424 [details].
<rdar://problem/69922171>