1559973 - Unify process types by extending GeckoProcessTypes enum with Content Types

Status: NEW

(Core :: IPC, enhancement, P3)

Description

[Tarek Ziadé (:tarek)]

Should we unify process types for clarity ?

Right now we have :
https://searchfox.org/mozilla-central/source/dom/ipc/ContentParent.h#52-59

And content subtypes:
https://searchfox.org/mozilla-central/source/xpcom/build/GeckoProcessTypes.h#26

My use case is to simplify ChromeUtils::RequestProcInfo() code

Comment 1

[Nathan Froyd [:froydnj]]

I think it is desirable to merge them. Content process checks would presumably need to use some function, then, to check for content process subtypes?

Comment 2

[Nika Layzell [:nika] (ni? for response)]

FYI Whatever solution is done for this will also have to handle the webIsolated=... process types used by fission-enabled browsers.

We currently put a dynamic string after the = with the site-origin of the website we're loading to ensure a separate process is used for each site-origin.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The priority flag is not set for this bug.
:jld, could you have a look please?

For more information, please visit auto_nag documentation.