Bug 1937836 Comment 4 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to Kaya [:kaya] from comment #0)
> With Fission coming up

Even when Fission is enabled, `PreallocatedProcessManager` only allocates processes until we hit the limit in `dom.ipc.processCount`. So at the moment we still won't have more than 2 content processes if we flip the pref.

Links to the code: [`PreallocatedProcessManagerImpl::CanAllocate()`](https://searchfox.org/mozilla-central/rev/aed442d01309eb6695c4ace1c4d8336e2d6cafc0/dom/ipc/PreallocatedProcessManager.cpp#273-278) calls [`ContentParent::IsMaxProcessCountReached(DEFAULT_REMOTE_TYPE)`](https://searchfox.org/mozilla-central/rev/aed442d01309eb6695c4ace1c4d8336e2d6cafc0/dom/ipc/ContentParent.cpp#721-726) which calls [`ContentParent::GetMaxProcessCount(DEFAULT_REMOTE_TYPE)`](https://searchfox.org/mozilla-central/rev/aed442d01309eb6695c4ace1c4d8336e2d6cafc0/dom/ipc/ContentParent.cpp#696)  which returns [`GetMaxWebProcessCount()`](https://searchfox.org/mozilla-central/rev/aed442d01309eb6695c4ace1c4d8336e2d6cafc0/toolkit/xre/nsAppRunner.cpp#6370-6381) which returns the value of the `dom.ipc.processCount` pref.
Revision 1 by
on 
(In reply to Kaya [:kaya] from comment #0)
> With Fission coming up

Even when Fission is enabled, `PreallocatedProcessManager` only allocates processes until we hit the limit in `dom.ipc.processCount`. So at the moment we still won't have more than 2 content processes if we flip the `dom.ipc.processPrelaunch.enabled` pref.

Links to the code: [`PreallocatedProcessManagerImpl::CanAllocate()`](https://searchfox.org/mozilla-central/rev/aed442d01309eb6695c4ace1c4d8336e2d6cafc0/dom/ipc/PreallocatedProcessManager.cpp#273-278) calls [`ContentParent::IsMaxProcessCountReached(DEFAULT_REMOTE_TYPE)`](https://searchfox.org/mozilla-central/rev/aed442d01309eb6695c4ace1c4d8336e2d6cafc0/dom/ipc/ContentParent.cpp#721-726) which calls [`ContentParent::GetMaxProcessCount(DEFAULT_REMOTE_TYPE)`](https://searchfox.org/mozilla-central/rev/aed442d01309eb6695c4ace1c4d8336e2d6cafc0/dom/ipc/ContentParent.cpp#696)  which returns [`GetMaxWebProcessCount()`](https://searchfox.org/mozilla-central/rev/aed442d01309eb6695c4ace1c4d8336e2d6cafc0/toolkit/xre/nsAppRunner.cpp#6370-6381) which returns the value of the `dom.ipc.processCount` pref.

Back to Bug 1937836 Comment 4