Closed
Bug 1085284
Opened 11 years ago
Closed 10 years ago
Implement URLSearchParams iterable<>
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: annevk, Assigned: baku)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file, 1 obsolete file)
5.46 KB,
patch
|
Details | Diff | Splinter Review |
URLSearchParams now gained the IDL iterable<> member. We should support it.
Reporter | ||
Updated•11 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Comment 1•11 years ago
|
||
It seems we don't support iteralbe<> in our webIDL codegen.
Depends on: 1085293
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → amarchesini
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8675439 -
Flags: review?(bzbarsky)
![]() |
||
Comment 3•10 years ago
|
||
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•10 years ago
|
||
Attachment #8675439 -
Attachment is obsolete: true
Assignee | ||
Comment 5•10 years ago
|
||
Comment 6•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Comment 7•10 years ago
|
||
Comment 8•9 years ago
|
||
Created:
https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams/keys
https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams/values
https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams/entries
Updated:
https://developer.mozilla.org/en-US/Firefox/Releases/44#Miscellaneous
https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams
Note that foreach() will be added as part of bug 1216751
Keywords: dev-doc-needed → dev-doc-complete
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•