Closed
Bug 959598
Opened 10 years ago
Closed 10 years ago
Investigate using a specialized routine to read sessionstore.js at startup
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: ttaubert, Unassigned)
References
Details
We're currently still in the early stage of looking at bug 959130 and assuming that the overhead of starting up the SessionWorker, waiting for it to processed the "read" message, and then returning the result introduced a regression in Firefox 25. SessionWorker.read() isn't doing anything special other than reading the file and returning a JSON string. We only read that file at startup and won't ever read it again. We should think about having a specialized method, maybe implemented in native code, that starts reading the session file early and can initialize sessionstore much earlier than we currently can.
Comment 1•10 years ago
|
||
This should be quite easy for an experienced developer. Or I'm willing to mentor some new contributor.
Whiteboard: [Async][mentor=Yoric][lang=c++][mentored but not simple]
Reporter | ||
Updated•10 years ago
|
Whiteboard: [Async][mentor=Yoric][lang=c++][mentored but not simple] → [mentor=Yoric][lang=c++][mentored but not simple]
Comment 2•10 years ago
|
||
(In reply to David Rajchenbach Teller [:Yoric] (please use "needinfo?") from comment #1) > This should be quite easy for an experienced developer. Or I'm willing to > mentor some new contributor. Is there any way I could help? I am new here and would like to work on my first patch, but I will need some help.
Comment 3•10 years ago
|
||
Thanks for offering, Rodrigo. We just realized that this is actually high-priority, so it turns out that this is not a good bug for mentoring. Sorry about that.
Whiteboard: [mentor=Yoric][lang=c++][mentored but not simple]
Reporter | ||
Updated•10 years ago
|
Blocks: fxdesktopbacklog
Whiteboard: [triage]
Updated•10 years ago
|
No longer blocks: fxdesktopbacklog
Whiteboard: [triage]
Comment 4•10 years ago
|
||
This is now being covered by Bug 961665 and Bug 970495.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•