Open Bug 1911376 Opened 3 months ago Updated 3 months ago

Consider renaming ChildID

Categories

(Core :: IPC, enhancement)

enhancement

Tracking

()

People

(Reporter: nika, Unassigned)

References

Details

In bug 1908725, we introduced a new ID type which is passed around in IPC alongside the pid, which is meant to be used in similar ways (i.e. as a public and easy to guess identifier for a specific process). The name ChildID was used for the initial implementation, as we already had a similar ChildID type in ContentParent/ContentChild, and as the more intuitively named GeckoProcessID would be too easily confused with the existing pid.

Unintuitively, this means that the parent process actually has a ChildID (of 0), which may be confusing. We should consider renaming it to something which is less tied to being a child process so it is less unintuitive that the parent process has a ChildID.

From some discussions with :smaug on matrix, the idea of IsolateID was proposed.

If you are open to suggestions: GeckoSubprocessID / SubprocessID or simply SubPID?

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