-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pass sourceText to OrdinaryFunctionCreate for event handlers #5514
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good, but we should remove the newline after the <dl class="switch">
. Although I guess I'd be open to doing that as a follow-up myself for the whole file (seems like something I might have missed the last time around).
This now has tests, which pass in Chrome Canary (not stable) and Firefox. We'll see what the bots say for Safari, and I'll file a bug if necessary. Edit: according to the test bots, Safari does not implement this; filed https://bugs.webkit.org/show_bug.cgi?id=211597. Adding @TimothyGu for an extra review if he has time, to double-check my event target logic. |
|
||
<dd> | ||
<dl class="switch"> | ||
<dt>If <var>eventHandler</var> is an <code data-x="handler-onerror">onerror</code> <span |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would go a bit further and say 'if name is "onerror
" '. The Window check looks good though.
…=testonly Automatic update from web-platform-tests Test the sourceText of event handlers Follows whatwg/html#5514. -- wpt-commits: af188628fbb56cbce2ba5c9ca418b5f465a64063 wpt-pr: 23467
…=testonly Automatic update from web-platform-tests Test the sourceText of event handlers Follows whatwg/html#5514. -- wpt-commits: af188628fbb56cbce2ba5c9ca418b5f465a64063 wpt-pr: 23467
…=testonly Automatic update from web-platform-tests Test the sourceText of event handlers Follows whatwg/html#5514. -- wpt-commits: af188628fbb56cbce2ba5c9ca418b5f465a64063 wpt-pr: 23467
…=testonly Automatic update from web-platform-tests Test the sourceText of event handlers Follows whatwg/html#5514. -- wpt-commits: af188628fbb56cbce2ba5c9ca418b5f465a64063 wpt-pr: 23467
Closes #5500.
(See WHATWG Working Mode: Changes for more details.)
/webappapis.html ( diff )