Closed Bug 328163 Opened 19 years ago Closed 18 years ago

Dump Session Data to Disk on Qualifying Events

Categories

(Firefox :: Tabbed Browser, enhancement)

2.0 Branch
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 2 alpha2

People

(Reporter: dietrich, Assigned: dietrich)

References

Details

(Keywords: fixed1.8.1, Whiteboard: 0d)

Implement support for dumping session data to disk on all qualifying events. Open issues: Which events qualify? What events does CrashRestore currently support? Should we do incremental saves? Does the complexity of incremental saves outweigh the performance hit of dumping entire session each time?
Blocks: 328154
Assignee: nobody → dietrich
Flags: blocking-firefox2+
Status: NEW → ASSIGNED
The current implementation (328159) collects session data from various events, and saves to disk at timed intervals. The supported events are basically: - window open & close - tab add, remove, load, close, input & select - "browser:purge-session-history" notification WRT to incremental saves: This makes no sense in the context of the current implementation; Session data is incrementally stored in memory, and the disk i/o is flow-controlled by a timer. This bug, as initially written, is overly broad. If someone thinks there's another scenario important enough to bypass the timer, this bug should be used to track it. (Currently, the only event that bypasses the timer is "browser:purge-session-history", upon which we purge the file.) This bug is essentially fixed by the patch in 328159, and I'll resolve it once that patch lands.
Whiteboard: 0d
This is implemented in the code that has been checked into both branch and trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Keywords: fixed1.8.1
You need to log in before you can comment on or make changes to this bug.