1785092 - Refactor script evaluation code to make use of a Realm class

Status: RESOLVED FIXED

(Remote Protocol :: WebDriver BiDi, task, P1)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

To serialize and deserialize complex Javascript objects the appropriate code in RemoteValue.jsm needs details about the realm (sandbox) it operates on. This includes not only the realm id but also the object cache that is used to store DebuggerObject instances and to generate their appropriate handleId.

With bug 1770480 there will be sandbox support for the script module, but it keeps known sandboxes within this module as simple maps and only sharing a sandbox (realm) id. Tying more data like the above will not evolve over time and will make it more complex to maintain and to share with other modules in the future.

As such each sandbox and the default window global should be wrapped within a Realm class so that each instance contains all the required data and can easily be shared via reference.

We need a base class for common code and most likely specific sub classes for individual realm types (window realm, sandbox realm, worker realm?, ...).

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1785092 - [remote] Added Realm class for Windows and Sandboxes.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3329e6a917a1
[remote] Added Realm class for Windows and Sandboxes. r=webdriver-reviewers,jdescottes

Comment 3

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/3329e6a917a1