Open Bug 1797039 Opened 2 years ago Updated 3 months ago

Separate the implementation of handle and mapping in SharedMemoryBasic

Categories

(Core :: IPC, task, P3)

Product:
Core
Component:
IPC
Type:
task

Tracking

()

People

(Reporter: nical, Unassigned)

References

Details

Bug 1795311 introduces new shmem classes with a distinction between handle and mapping. These are currently implemented on top of a single SharedMemoryBasic class which leads to some awkwardness and limitations.

Can you give specific examples of the awkwardness and limitations that should be addressed?

Flags: needinfo?(nical.bugzilla)

This is completely paged out of my brain, so off hands I'm not sure. I suspect that Nika took care of it in this commit: https://hg.mozilla.org/mozilla-central/rev/3bba8dcd16a0dc2a928b9042ac4f6c8f9c5db0db#l3.12

I'm referring to clone-then-close danse that the previous version of the code used to do. Nika's commit notes that the current code is "not the ideal form of this API" but short of remembering what I thought would be the best thing back then, it looks reasonable to me.

Flags: needinfo?(nical.bugzilla)
You need to log in before you can comment on or make changes to this bug.