1913377 - Comment `allowedScope` better in structured clone

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task)

Description

[Steve Fink [:sfink] [:s:]]

JSStructuredCloneReader::allowedScope is confusing, especially given the existence of all the funky JS::StructuredCloneScope values like Unassigned, UnkonwnDestination, and DifferentProcessForIndexedDB. But they can't just be disallowed, since they're all used. Fortunately, they are all resolved down to one of the simple values during initialization.

Document and assert this.

Comment 1

[Steve Fink [:sfink] [:s:]]

Attachment: Bug 1913377 - Comment and assert that `allowedScope` has a very limited set of values after readHeader()

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6be560f613e3
Comment and assert that `allowedScope` has a very limited set of values after readHeader() r=spidermonkey-reviewers,iain

Comment 3

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/6be560f613e3