1734166 - Investigate using structuredClone for local transport instead of freezing packets

Status: RESOLVED WONTFIX

(DevTools :: Framework, task)

Description

[Nicolas Chevobbe [:nchevobbe]]

In local transport, we freeze object because we don't want to serialize the packets for perf reasons, but we don't want to risk modifying server objects from the client.

https://searchfox.org/mozilla-central/rev/9bc5dcea99c59dc18eae0de7064131aa20cfbb66/devtools/shared/transport/local-transport.js#53

We might want to replace deepFreeze with structuredClone so we don't share object instances between client and server.
We should check if this does have any performance impact.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Here's some DAMP results: https://treeherder.mozilla.org/perfherder/comparesubtest?originalProject=try&newProject=try&newRevision=cb2cd8f9816a49523e793aa2d1ae906274503d08&originalSignature=3130994&newSignature=3130994&framework=12&originalRevision=5efe0d3c1d9a50322d8189a79c44512d37519847&page=1&showOnlyImportant=1

This has significant performance regression, so I don't think we'll move on with this.