Closed
Bug 960572
Opened 11 years ago
Closed 6 years ago
[meta][Session Restore] Provide a clean API for accessing data object
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
INACTIVE
People
(Reporter: Yoric, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Keywords: meta)
Attachments
(1 file)
At the moment, saving proceeds as follows:
1. collect data;
2. build big object;
3. serialize object;
4. send to worker for writing.
I would like to replace step 2 by something a little more abstract that will later let us rework the back-end without having to rewrite everything.
Attachment #8361116 -
Flags: feedback?(ttaubert)
Comment 1•11 years ago
|
||
Comment on attachment 8361116 [details]
Rough sketch of a possible API
Yes, that seems like an interesting thing to have and it would probably make the code a little more explicit about what just happened and why we're updating state. It seems like a big project though so I wouldn't really want to tackle that until we're sure we're doing something that would profit from differential updates.
Attachment #8361116 -
Flags: feedback?(ttaubert) → feedback+
Reporter | ||
Comment 2•10 years ago
|
||
Turning this into a meta-bug, as we'll be doing this piece-wise.
Keywords: meta
Summary: [Session Restore] Provide a clean API for accessing data object → [meta][Session Restore] Provide a clean API for accessing data object
Reporter | ||
Comment 3•10 years ago
|
||
Note that the ongoing work is about providing access to the state (i.e. replacing `getBrowserState` & co.), not changing state in a way that will be mirrored automatically by the DOM (i.e. we are not replacing `setBrowserState` & co.).
Blocks: Session_managers
No longer blocks: Session_managers
Comment 4•6 years ago
|
||
I don't think we'll end up doing this particular piece, since we already have gained a WebExtension API for sessionstore.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
You need to log in
before you can comment on or make changes to this bug.
Description
•