Introduce DocumentAndElementEventHandlers

RESOLVED FIXED in Firefox 53

Status

()

Core
DOM
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: bz, Assigned: bz)

Tracking

Trunk
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

See https://html.spec.whatwg.org/multipage/webappapis.html#documentandelementeventhandlers

This doesn't really change web-visible behavior; just aligns our IDL more closely with the spec.
Created attachment 8826043 [details] [diff] [review]
Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec
Attachment #8826043 - Flags: review?(bkelly)

Comment 2

11 months ago
Comment on attachment 8826043 [details] [diff] [review]
Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec

Review of attachment 8826043 [details] [diff] [review]:
-----------------------------------------------------------------

Can you point out where the SVGElement spec for these event handlers is?

::: dom/webidl/SVGElement.webidl
@@ +28,5 @@
>    [Throws] void blur();
>  };
>  
>  SVGElement implements GlobalEventHandlers;
> +SVGElement implements DocumentAndElementEventHandlers;

I don't see this in the spec.  Maybe I have the wrong link.  Looking at:

https://svgwg.org/svg2-draft/types.html#InterfaceSVGElement
Attachment #8826043 - Flags: review?(bkelly)
Comment on attachment 8826043 [details] [diff] [review]
Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec

The SVG spec situation here is a mess.  See https://www.w3.org/Graphics/SVG/WG/track/actions/3283 which hasn't happened yet (and is linked from <https://svgwg.org/svg2-draft/interact.html#SVGEvents>), but in general their intent is that the set of event handlers on HTML and SVG elements be identical.
Attachment #8826043 - Flags: review?(bkelly)

Comment 4

11 months ago
Comment on attachment 8826043 [details] [diff] [review]
Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec

Review of attachment 8826043 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks.
Attachment #8826043 - Flags: review?(bkelly) → review+

Comment 5

11 months ago
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/426c5f5536bf
Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec.  r=bkelly

Comment 6

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/426c5f5536bf
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.