Open Bug 1689446 Opened 4 years ago Updated 1 year ago

Only broadcast SharedMap changes to preallocated processes when they're consumed

Categories

(Core :: DOM: Content Processes, task, P3)

task

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.

Severity: -- → N/A
Priority: -- → P3
Fission Milestone: --- → MVP

This is a perf optimization and doesn't need to block Fission MVP. Moving to Fission Future.

Fission Milestone: MVP → Future
You need to log in before you can comment on or make changes to this bug.