WebIDL bindings should support Constructor(optional (USVString or something) foo = "")

RESOLVED FIXED in Firefox 53

Status

()

defect
RESOLVED FIXED
3 years ago
5 months ago

People

(Reporter: baku, Assigned: bzbarsky)

Tracking

50 Branch
mozilla53
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

No description provided.
Blocks: 1330692, 1330678
This is actually only a problem with a string literal default value for a union-type argument containing USVString.  If the argument type is USVString we coerce the literal to be DOMString-typed, so there is no problem.  But in the union case we have to give the literal the type of the thing it matched inside the union, which is what causes the issue.
Summary: WebIDL bindings should support Constructor(optional USVString foo = "") → WebIDL bindings should support Constructor(optional (USVString or something) foo = "")
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #8826259 - Flags: review?(kyle) → review+
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b75e432a3b77
Support string default values for unions containing USVString.  r=qdot
https://hg.mozilla.org/mozilla-central/rev/b75e432a3b77
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Blocks: 1331580
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.