Open Bug 1981641 Opened 6 months ago Updated 6 months ago

Split process_util.h into basic types vs. launch-related declarations

Categories

(Core :: IPC, enhancement)

Product:
Core
Component:
IPC
Type:
enhancement

Tracking

()

People

(Reporter: jld, Unassigned)

References

(Blocks 1 open bug)

Details

Inspired by bug 785103, something that's been mildly annoying me every now and then over the years: process_util.h is included into a lot of surprising things because of the process id/handle types, which are used by IPDL things, which end up transitively included all over the place. But, a lot of process_util.h is specific to process launching, which is used only in a relatively small amount of IPC-specific code.

…I think. I haven't looked at this in a while, and there might be dependency chains like LaunchOptions → ContentParent → half of the tree, but if so I think those could be handled with forward declarations.

For reference, Chromium made a similar change in 2013; see https://crbug.com/242290

You need to log in before you can comment on or make changes to this bug.