Only broadcast SharedMap changes to preallocated processes when they're consumed
Categories
(Core :: DOM: Content Processes, task, P3)
Tracking
()
Fission Milestone | Future |
People
(Reporter: kmag, Unassigned)
References
(Blocks 1 open bug)
Details
Whenever a SharedMap entry is updated in the parent process, we (eventually) broadcast an updated copy of the map to all content processes, including preallocated ones which are sitting idle. In principle, though, there shouldn't be any need to notify preallocated processes of the changes until they're actually consumed from the pool, which would avoid unnecessary wake-ups.
In practice, this probably isn't especially critical, since SharedMap updates are relatively infrequent. But even so, it would be nice.
There may be other code which follows similar patterns that could cause much bigger issues, though, so we should probably audit other callers of ContentParent::GetAll()
and figure out which ones really need to be lazified.
Updated•4 years ago
|
Updated•4 years ago
|
Comment 1•3 years ago
|
||
This is a perf optimization and doesn't need to block Fission MVP. Moving to Fission Future.
Description
•