Closed
Bug 1360874
Opened 8 years ago
Closed 8 years ago
Self-support should wait for an idle callback before delayed initialization
Categories
(Firefox :: General, enhancement)
Firefox
General
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: kmag, Unassigned)
References
(Blocks 1 open bug)
Details
Self-support currently waits for 5 seconds after sessionstore-windows-restored before running its delayed initialization code. There's no guarantee that this is a good time for it to run, though, so it should really wait for an idle callback instead.
Updated•8 years ago
|
Blocks: photon-performance-triage
Whiteboard: [photon-performance]
Updated•8 years ago
|
Flags: qe-verify?
Priority: -- → P2
Comment 1•8 years ago
|
||
We're planning to remove self-support in favour of SHIELD / normandy. Not sure how useful it is to invest in it much in the meantime. Mike, can you clarify what the latest schedule for those plans looks like?
Flags: needinfo?(mcooper)
Reporter | ||
Comment 2•8 years ago
|
||
Just waiting for an idle callback after the timer expires is a pretty minor investment, which might be a non-trivial startup perf win...
Comment 3•8 years ago
|
||
We shipped the Shield system add-on to release today at 100%. The system add-on disables self-support. so this shouldn't affect any more users.
Also, I think we're ready to remove self-support now, since it's replacement has shipped.
Flags: needinfo?(mcooper)
Updated•8 years ago
|
Blocks: photon-perf-jank
Updated•8 years ago
|
No longer blocks: photon-performance-triage
Comment 5•8 years ago
|
||
Discussed in team meeting and decision was to mark as wontfix. Team will track dependency (Bug 1361578) as part of the Photon Performance Project.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: qe-verify?
Priority: P2 → --
Resolution: --- → WONTFIX
Whiteboard: [photon-performance]
Comment 6•8 years ago
|
||
For additional context, this was WONTFIX'd because the self-support system is going away.
You need to log in
before you can comment on or make changes to this bug.
Description
•