Implement URLSearchParams iterable<>

RESOLVED FIXED in Firefox 44

Status

()

defect
RESOLVED FIXED
5 years ago
3 months ago

People

(Reporter: annevk, Assigned: baku)

Tracking

({dev-doc-complete})

unspecified
mozilla44
x86
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox44 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

5 years ago
URLSearchParams now gained the IDL iterable<> member. We should support it.
Reporter

Updated

5 years ago
Keywords: dev-doc-needed
Assignee

Comment 1

5 years ago
It seems we don't support iteralbe<> in our webIDL codegen.
Depends on: 1085293

Updated

5 years ago
Blocks: 887836
Assignee

Updated

4 years ago
Assignee: nobody → amarchesini
Assignee

Comment 2

4 years ago
Posted patch iterable1.patch (obsolete) — Splinter Review
Attachment #8675439 - Flags: review?(bzbarsky)
Comment on attachment 8675439 [details] [diff] [review]
iterable1.patch

>+      is(key.value, v.toString(), "Correcy Key iterator: " + v.toString());

s/Correcy/Correct/, and same elsewhere in this test.

You probably want to assert that key.done/value.done/entry.done are false in the loop.

r=me
Attachment #8675439 - Flags: review?(bzbarsky) → review+
Assignee

Comment 4

4 years ago
Attachment #8675439 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/c1ff09b2e8fa
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Component: DOM → DOM: Core & HTML
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.