All content processes receive an IPC whenever the new tab page is updated
Categories
(Toolkit :: Places, defect)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox113 | --- | fixed |
People
(Reporter: florian, Assigned: florian)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
See this profile of Firefox being idle for multiple hours: https://share.firefox.dev/425QUGf
Whenever the new tab page is updated, the privileged content process sends an PContent::Msg_StartVisitedQueries IPC to the parent, that then sends PContent::Msg_NotifyVisited messages to all content processes (including preallocated content processes and the webextension process).
Is this the expected behavior? Was bug 1714614 meant to avoid this?
|
||
Comment 1•2 years ago
|
||
What URI(s) are we sending in NotifyVisited? Whenever we visit a new URI we need to broadcast that to all processes.
|
Assignee | |
Comment 2•2 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #1)
What URI(s) are we sending in
NotifyVisited?
Most of the time, none.
|
|
Assignee | |
Comment 3•2 years ago
|
||
|
||
Updated•2 years ago
|
|
|
Assignee | |
Comment 4•2 years ago
|
||
(In reply to Florian Quèze [:florian] from comment #2)
(In reply to Emilio Cobos Álvarez (:emilio) from comment #1)
What URI(s) are we sending in
NotifyVisited?Most of the time, none.
Example profile (without the fix, but with extra debug markers) showing this: https://share.firefox.dev/3ZRjt8N
Profile with the fix and the same debug markers: https://share.firefox.dev/3ZSWrOU We no longer wake-up the preallocated content processes when updating the cached new tab page in the privileged content process.
|
|
Assignee | |
Updated•2 years ago
|
|
||
Comment 6•2 years ago
|
||
| bugherder | ||
Description
•