Closed Bug 1350254 Opened 3 years ago Closed 5 months ago

Use IDL syntax for serializable and transferable objects

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: annevk, Assigned: bzbarsky)

References

(Blocks 2 open bugs)

Details

Attachments

(10 files)

47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
HTML now has [Serializable] and [Transferable] syntax to annotate platform objects with that can be serialized and transfered.

We should implement it since it was our suggestion.
Priority: -- → P3
Depends on: 1456604
No longer depends on: 1456604
Component: DOM → DOM: Core & HTML

bz, do you want to take this bug?

Assignee: amarchesini → nobody
Flags: needinfo?(bzbarsky)

Sure. I'm working on the [Serializable] bit; not sure about [Transferable] yet.

Assignee: nobody → bzbarsky
Flags: needinfo?(bzbarsky)
Flags: needinfo?(bzbarsky)

The spec doesn't say to do this, but I think we should. See
https://github.com/w3c/webcrypto/issues/222

Flags: needinfo?(bzbarsky)
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d27f30e55f4e
part 1.  Add support for [Serializable] in the Web IDL parser and code generator.  r=baku,mccr8
https://hg.mozilla.org/integration/autoland/rev/0a7cdfb04b87
part 2.  Switch ImageData to [Serializable].  r=baku
https://hg.mozilla.org/integration/autoland/rev/f8b0c0b1aa91
part 3.  Switch DOMQuad to [Serializable].  r=baku
https://hg.mozilla.org/integration/autoland/rev/215283c818b4
part 4.  Switch DOMPoint and DOMPointReadOnly to [Serializable].  r=baku
https://hg.mozilla.org/integration/autoland/rev/c5f545a54a53
part 5.  Switch DOMRect and DOMRectReadOnly to [Serializable].  r=baku
https://hg.mozilla.org/integration/autoland/rev/9afaa7c609ea
part 6.  Switch DOMMatrix and DOMMatrixReadOnly to [Serializable].  r=baku
https://hg.mozilla.org/integration/autoland/rev/e6329e158fc1
part 7.  Consolidate ReadString/WriteString helpers for structured serialization.  r=baku
https://hg.mozilla.org/integration/autoland/rev/e02c73e7bd4b
part 8.  Switch CryptoKey to [Serializable].  r=baku
https://hg.mozilla.org/integration/autoland/rev/954a6de2b937
part 9.  Switch URLSearchParams to [Serializable].  r=baku
https://hg.mozilla.org/integration/autoland/rev/96e19ef1e191
part 10.  Switch RTCCertificate to [Serializable].  r=baku
Blocks: 1562064
Blocks: 1562065
Blocks: 1562066

I filed bug 1562066 on non-fully-serializable objects and bug 1562065 on [Transferable].

You need to log in before you can comment on or make changes to this bug.