Closed Bug 2015391 Opened 6 months ago Closed 4 months ago

Recently closed tabs persist when Browsing history is selected in Delete browsing data on quit

Categories

(Firefox for Android :: History, defect)

Firefox 149
All
Android
defect

Tracking

()

VERIFIED FIXED
151 Branch
Tracking Status
firefox149 --- wontfix
firefox150 --- wontfix
firefox151 --- verified
firefox152 --- verified

People

(Reporter: denyshon, Assigned: alastun)

References

Details

(Whiteboard: [fxdroid][group4])

Attachments

(1 file)

Steps to reproduce

  1. Enable "Delete browsing data on quit".
  2. Make sure "Browsing history" is checked.
  3. Open several tabs and close some of them.
  4. Quit Firefox.
  5. Reopen Firefox.

Expected behavior

The "Recently closed tabs" section in History is empty, because the whole history has been cleared.

Actual behavior

The "Recently closed tabs" section content persists.

Device information

  • Firefox version: 149.0a1
  • Android device model: Redmi Note 7 Pro
  • Android OS version: 10

Any additional information?

Initially brought up here.

We likely are missing the clean-up logic for Recently Closed for this specific case.

This is the function we should invoke during this browsing history tear down operation.

https://searchfox.org/firefox-main/source/mobile/android/android-components/components/feature/recentlyclosed/src/main/java/mozilla/components/feature/recentlyclosed/RecentlyClosedTabsStorage.kt#78

Severity: -- → S3
Whiteboard: [fxdroid][group4]

Actually I think this is a regression in Firefox for Android 148.0, althugh it is being reported against 149. I am seeing this exact issue starting with 148.0. Previously, using Firefox for Android 147.0.4, all 'recently closed tabs' were cleared when I quit Firefox, but starting with 148.0 they are still there next time I start. (I do have Delete browsing data on quit set to ON, and do have Browsing History and Open Tabs checked.)

See Also: → 1889742
Duplicate of this bug: 2023626
Assignee: nobody → alastun

Use parallel execution (supervisorScope) to prevent SQLite deletion from blocking BrowserStore dispatches during app termination.

Pushed by alastun@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/ad67575d7ed5 https://hg.mozilla.org/integration/autoland/rev/84731130f194 Persistent Recently Closed Tabs by implementing parallel history deletion and RAM state clearing r=android-reviewers,rebecatudor273,twhite
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch

Could we request an uplift please?

(In reply to Anthony from comment #7)

Could we request an uplift please?

We could, but this fix is only part of two major bugs in this category, the latter being more important: [link]. I was thinking we should fix both and then do the uplift.

(The link above is broken, but it should lead to bug 1992645)

Since nightly and release are affected, beta will likely be affected too.
For more information, please visit BugBot documentation.

The patch landed in nightly and beta is affected.
:alastun, is this bug important enough to require an uplift?

For more information, please visit BugBot documentation.

Flags: needinfo?(alastun)
Flags: needinfo?(alastun)
Regressions: 2033285
Duplicate of this bug: 2026108

QA could you please verify if this bug was fixed? There seems to be users who still get the issue. The patch is already in Beta, so both Nightly and Beta should be tested. Thank you!

Flags: qe-verify+

Verified as fixed on the latest Nightly 152.0a1 and on the latest Beta (151.0b2) builds.

The "Recently closed tabs" section in History is empty, because the whole history has been cleared.

Devices used:

  • Samsung Galaxy S25 Ultra (Android 16);
  • Xiaomi Mi11 Lite (Android 11).
Status: RESOLVED → VERIFIED
Duplicate of this bug: 2035566
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: