Make improvements to the SessionStore#CloseTab API




4 years ago
4 months ago


(Reporter: mfinkle, Unassigned)


Firefox Tracking Flags

(Not tracked)


Two things come to mind:
1. Convert the *CloseTab methods take an index and not a JSON data literal. This matches Desktop and makes a lot more sense.
2. Remove "old" closed tab data. Desktop removes closed tab data after a given time.

Looks like #2 is done in an Idle handler:

#2 would require changing the Fennec closedTab JSON structure a little, but it would reduce the size of the file.
Can't say anything off-hand about #1, but the Desktop implementation for #2 now lives at

The default value on Desktop is to cleanup old "Recently closed tabs" after two weeks, so based on that even moderate usage would likely hit our current value of browser.sessionstore.max_tabs_undo first, but it might still be an interesting idea.
Component: General → Session Restore
OS: Mac OS X → Android
Priority: -- → P3
Hardware: x86 → All
Re-triaging per

Needinfo :susheel if you think this bug should be re-triaged.
Priority: P3 → P5
You need to log in before you can comment on or make changes to this bug.