1330692 - URLSearchParams constructor is not supported by our webidl codegen

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

[Andrea Marchesini [:baku]]

Recently URLSearchParams CTOR is changed:

[Constructor(optional (sequence<sequence<USVString>> or record<USVString, USVString> or USVString) init = ""), ...

but record is not supported, and also without it, 'Constructor(optional (sequence<sequence<USVString>> or USVString) init = "")' gives syntax error.

Comment 1

[Boris Zbarsky [:bzbarsky]]

record is kinda supported as MozMap.  But it doesn't have quite the same semantics (e.g. is not order-preserving).  We should probably modify MozMap to be named "record" and change its implementation to match the spec.

> and also without it, 'Constructor(optional (sequence<sequence<USVString>> or USVString) init = "")'
> gives syntax error

I don't get a syntax error here.  I do get a codegen error because we don't support USVString as the type of the default value in handleDefaultStringValue.  That should be pretty simple to fix.

Comment 2

[Boris Zbarsky [:bzbarsky]]

When you add this, please add a test that the _key_ values are getting the USVString treatment.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Long since fixed when the dependencies got fixed.