[go: up one dir, main page]
Closed Bug 1331580 Opened 8 years ago Closed 8 years ago

Construct URLSearchParams from record<>

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla54
Tracking Flags:
Tracking Status
firefox54 --- fixed

People

(Reporter: baku, Assigned: baku)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

ucs.patch
4.61 KB, patch
qdot
: review+
Details | Diff | Splinter Review
The record constructor test had a small bug that is being fixed here: https://github.com/w3c/web-platform-tests/pull/4647.
Does that mean that once the above PR goes in (which seems to have happened), this bug is invalid?
Flags: needinfo?(annevk)
No, our IDL doesn't use record<> yet as that isn't implemented per dependency bug 1330699. You'll note that we still fail those tests.
Flags: needinfo?(annevk)
ok, thanks for the clarification!
Priority: -- → P3
Keywords: dev-doc-needed
Attached patch ucs.patch — — Splinter Review
Assignee: nobody → amarchesini
Attachment #8842843 - Flags: review?(kyle)
Attachment #8842843 - Flags: review?(kyle) → review+
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/ee9c3cd7d55f Construct URLSearchParams from record<>, r=qdot
https://hg.mozilla.org/mozilla-central/rev/ee9c3cd7d55f
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Did this add the test I asked for in bug 1330692 comment 2?
Flags: needinfo?(amarchesini)
Bug 1352379
Flags: needinfo?(amarchesini)
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: