Open Bug 1454816 Opened 6 years ago Updated 1 month ago

Flatten base::SharedMemory into SharedMemoryBasic

Categories

(Core :: IPC, enhancement, P3)

60 Branch
enhancement

Tracking

()

People

(Reporter: jld, Unassigned)

References

Details

base::SharedMemory and mozilla::ipc::SharedMemoryBasic are basically the same thing, with SharedMemoryBasic_chromium.cpp being a thin glue layer.

We could remove one of the (confusingly many) shared memory classes by: translating base::SharedMemory into Mozilla style, changing its two implementations into SharedMemoryBasic implementations, adjusting other base::SharedMemory clients to use SharedMemoryBasic instead, and removing base::SharedMemory.

(At some point we may also want to reconsider the name ipc/glue/, because at this point a lot of it really isn't glue for anything.)
See Also: → 1454819
Priority: -- → P3

One complication here is that we'll probably need POSIX and Mach shared memory to coexist on macOS, at least in the short term.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.