Implement CustomElementsRegistry get function

RESOLVED FIXED in Firefox 52

Status

()

defect
P2
normal
RESOLVED FIXED
3 years ago
4 months ago

People

(Reporter: jdai, Assigned: jdai)

Tracking

({dev-doc-complete})

unspecified
mozilla52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

(Whiteboard: dom-ce-m1)

Attachments

(1 attachment, 3 obsolete attachments)

According to session 4.13.4[1], we need to implement CustomElementsRegistry get function.


[1] https://html.spec.whatwg.org/multipage/scripting.html#dom-customelementsregistry-get
(I'm marking all the Custom Elements bugs as "backlog" but that's just to indicate they're not something we're fixing urgently and no comment on priority or anything.)
Whiteboard: btpp-backlog
Assignee: nobody → jdai
Depends on: 1275835
Posted patch wip, v1 (obsolete) — Splinter Review
Priority: -- → P2
Whiteboard: btpp-backlog → dom-ce-m1
Attachment #8773697 - Attachment is obsolete: true
Attachment #8789736 - Flags: feedback?(echen)
Comment on attachment 8789736 [details] [diff] [review]
Bug 1275838 - Implement CustomElementsRegistry get function.

Review of attachment 8789736 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #8789736 - Flags: feedback?(echen) → feedback+
Attachment #8789736 - Flags: review?(wchen)
Attachment #8789736 - Flags: review?(wchen) → review+
Keywords: checkin-needed
need dom peer review: remote: WebIDL file dom/webidl/CustomElementsRegistry.webidl altered in changeset f0c053165318 without DOM peer review
Flags: needinfo?(jdai)
Keywords: checkin-needed
Comment on attachment 8791113 [details] [diff] [review]
Bug 1275838 - Implement CustomElementsRegistry get function. r=wchen

I forgot to ask DOM peer review. 
Hi Olli, 
Could you help to review my patch? Thank you.
Flags: needinfo?(jdai)
Attachment #8791113 - Flags: review+ → review?(bugs)
Comment on attachment 8791113 [details] [diff] [review]
Bug 1275838 - Implement CustomElementsRegistry get function. r=wchen

r+ for the .webidl
Attachment #8791113 - Flags: review?(bugs) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a2d8c1189368
Implement CustomElementsRegistry get function. r=smaug, r=wchen
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a2d8c1189368
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
I've made sure this method is documented:
https://developer.mozilla.org/en-US/docs/Web/API/CustomElementRegistry/get

I've also added a note to the Fx52 release notes: 
https://developer.mozilla.org/en-US/Firefox/Releases/52#DOM_HTML_DOM

Let me know if this is OK. Thanks!
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.