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)

defect
Not set
normal

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.
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]
Whiteboard: [Async][mentor=Yoric][lang=c++][mentored but not simple] → [mentor=Yoric][lang=c++][mentored but not simple]
(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.
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]
Whiteboard: [triage]
No longer blocks: fxdesktopbacklog
Whiteboard: [triage]
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.