963321 - Support returning cached dictionaries from JS-implemented WebIDL

Status: RESOLVED FIXED

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

Description

[Blake Kaplan (:mrbkap) (inactive)]

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.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: part 1. Add WebIDL parser support for having [Cached] dictionary attributes.

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.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: part 2. Add codegen support for [Cached] dictionary attributes.

Comment 3

[Boris Zbarsky [:bzbarsky]]

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.

Comment 4

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

I assume we will have something testing this from JS.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Once bug 886110 is fixed, presumably.

Comment 6

[Boris Zbarsky [:bzbarsky]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2d8c7d793f45
https://hg.mozilla.org/integration/mozilla-inbound/rev/fdb5cb962614

Comment 7

[Boris Zbarsky [:bzbarsky]]

Documented at https://developer.mozilla.org/en-US/docs/Mozilla/WebIDL_bindings#Cached

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2d8c7d793f45
https://hg.mozilla.org/mozilla-central/rev/fdb5cb962614