Closed
Bug 1266223
Opened 9 years ago
Closed 8 years ago
Recently closed tabs are not persisted across app kills
Categories
(Firefox for Android Graveyard :: Awesomescreen, defect)
Firefox for Android Graveyard
Awesomescreen
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: Margaret, Assigned: JanH)
References
Details
Attachments
(1 file)
MozReview Request: Bug 1266223 - Add logging for session store closed tab data messaging. r=margaret
(deleted),
text/x-review-board-request
|
Margaret
:
review+
|
Details |
When Fennec is killed, my recently closed tabs are not restored in my "Recent Tabs" panel.
Additionally, because I have my browser set to always restore tabs (the default), the items in "Tabs from last time" are really old, and not particularly useful.
In this bug, I think we should make sure recently closed tabs are persisted across app runs, but in the future I think we should consider hiding the "Tabs from last time" section for people who have their tabs set to always restore.
Assignee | ||
Comment 1•9 years ago
|
||
I'm afraid I can't reproduce this (in FF44 and today's Nightly):
- I've closed some tabs
- made sure they appear under Recent Tabs
- switched away from Firefox and killed it by swiping it recently the task switcher
- relaunched Firefox and all my recently closed tabs where present, as large as life and twice as natural
In the session store code, we attach the closed tab data to the window during restore and then later save it back out as needed, so the data shouldn't get lost. The only thing that should be able to go wrong would be a corrupted session store file, which would probably be bug 1190627.
As for the tabs from last time, the easiest solution seems deleting out-of-date sessionstore.bak files during startup if we're restoring tabs, which automatically hides the "tabs from last time" section.
Assignee | ||
Updated•9 years ago
|
Updated•9 years ago
|
Assignee | ||
Comment 2•9 years ago
|
||
Looking a bit closer, one thing that theoretically could go wrong is that the session store might not be quite ready when the "ClosedTabs:StartNotifications" notification arrives.
Seems like I should add some logging for that, too.
Assignee | ||
Comment 3•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/49229/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/49229/
Attachment #8746026 -
Flags: review?(margaret.leibovic)
Reporter | ||
Comment 4•9 years ago
|
||
Comment on attachment 8746026 [details]
MozReview Request: Bug 1266223 - Add logging for session store closed tab data messaging. r=margaret
https://reviewboard.mozilla.org/r/49229/#review46105
Attachment #8746026 -
Flags: review?(margaret.leibovic) → review+
Assignee | ||
Comment 5•9 years ago
|
||
Keywords: checkin-needed,
leave-open
Keywords: checkin-needed
Comment 7•9 years ago
|
||
bugherder |
Assignee | ||
Comment 8•8 years ago
|
||
Margaret, can you reproduce this with logging enabled?
Flags: needinfo?(margaret.leibovic)
Reporter | ||
Comment 9•8 years ago
|
||
(In reply to Jan Henning [:JanH] from comment #8)
> Margaret, can you reproduce this with logging enabled?
I haven't seen this happen recently, and I can't reproduce when trying right now.
I don't often look at my recent tabs panel, but I can try to look more closely to see if this happens again.
Flags: needinfo?(margaret.leibovic)
Assignee | ||
Comment 10•8 years ago
|
||
I've seen this myself now. Unfortunately, I still don't know what exactly went wrong, but my suspicion is that the problem is not in persisting the closed tab data, but in sending it from the session store to the Java UI. After switching to a different tab and back to about:home, which triggered a cycle of ClosedTabs:StopNotifications and ClosedTabs:StartNotifications, the closed tabs were properly transmitted and reappeared in the recent tabs panel.
Assignee | ||
Comment 11•8 years ago
|
||
Haven't really seen this recently.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Comment 12•7 years ago
|
||
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•