[Webkit-unassigned] [Bug 166973] Construct URLSearchParams from array or object

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jan 17 14:29:00 PST 2017


https://bugs.webkit.org/show_bug.cgi?id=166973

--- Comment #4 from Alex Christensen <achristensen at apple.com> ---
I have four issues with this so far:
1) The spec doesn't specify the order we iterate the record in.  Our implementation is a HashMap<String, String>.  Is this wrong?
2) There is no test with a sequence of one USVString.  A *very* poor implementation might go out of bounds here.  I think it's worth testing.
3) There probably isn't a test for setting or appending a '+' in an existing URLSearchParams.  Lots of places we replace a '+' with a ' ', but my implementation doesn't in set or append, but there seems to be such a test with the record constructor.  urlsearchparams-stringifier.html has a contradictory test verifying that '+' is stringified to '+'.
4) The HashMap<String, String> I get from the values you see in the patch I'm about to upload.  Is that a problem in our record implementation?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170117/037b03c2/attachment.html>


More information about the webkit-unassigned mailing list