Closed Bug 1966134 Opened 1 year ago Closed 1 year ago

Replace nsIArrayBufferInputStream with a manual writing to the bulk stream

Categories

(DevTools :: Performance Tools (Profiler/Timeline), enhancement)

Product:
DevTools
Component:
Performance Tools (Profiler/Timeline)
Type:
enhancement

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: julienw, Unassigned)

References

Details

Since Bug 1581963 we use the bulk protocol to transfer the profile data. But on the actor side, this uses an nsIArrayBufferInputStream which does a memory copy. Instead we should write the data directly to the bulk stream with the help of the BinaryOutputStream.
We can probably get a lot of inspiration from AsyncStreamToArrayBufferCopier implemented in the front. It would be good to gather these copiers to a shared file (possibly even to stream-utils.js) so that they could possibly be reused elsewhere later.

Did it in bug 1581963 after all.

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.