Closed
Bug 963321
Opened 10 years ago
Closed 10 years ago
Support returning cached dictionaries from JS-implemented WebIDL
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla30
People
(Reporter: mrbkap, Assigned: bzbarsky)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(2 files)
4.35 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
6.83 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
In the wifi API, we want to be able to return an object that can update itself, but is otherwise readonly to DOM content. We want to be able to use a dictionary to do this and bz informs me that this will be possible with a small change to codegen.
Reporter | ||
Updated•10 years ago
|
Flags: needinfo?(bzbarsky)
![]() |
Assignee | |
Comment 1•10 years ago
|
||
We don't know at handleExtendedAttribute time what the identifier will map to, so have to do the check that [Frozen] is not being misused later.
Attachment #8364806 -
Flags: review?(khuey)
![]() |
Assignee | |
Updated•10 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
![]() |
Assignee | |
Comment 2•10 years ago
|
||
Attachment #8364808 -
Flags: review?(khuey)
![]() |
Assignee | |
Comment 3•10 years ago
|
||
I filed bug 963382 on clearing the cached values from JS, since we don't have a way of doing that for readonly properties yet. That part isn't really dictionary-specific.
Flags: needinfo?(bzbarsky)
Whiteboard: [need review]
![]() |
Assignee | |
Updated•10 years ago
|
Blocks: ParisBindings
Attachment #8364806 -
Flags: review?(khuey) → review+
Attachment #8364808 -
Flags: review?(khuey) → review+
I assume we will have something testing this from JS.
![]() |
Assignee | |
Comment 5•10 years ago
|
||
Once bug 886110 is fixed, presumably.
![]() |
Assignee | |
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/2d8c7d793f45 https://hg.mozilla.org/integration/mozilla-inbound/rev/fdb5cb962614
Flags: in-testsuite+
Whiteboard: [need review]
Target Milestone: --- → mozilla30
![]() |
Assignee | |
Comment 7•10 years ago
|
||
Documented at https://developer.mozilla.org/en-US/docs/Mozilla/WebIDL_bindings#Cached
Keywords: dev-doc-complete
Comment 8•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2d8c7d793f45 https://hg.mozilla.org/mozilla-central/rev/fdb5cb962614
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•