Closed Bug 1859358 Opened 1 year ago Closed 9 months ago

Remove hardcoded limit of 25 recently closed tabs in Firefox View

Categories

(Firefox :: Firefox View, task, P2)

task

Tracking

()

RESOLVED FIXED
123 Branch
Tracking Status
firefox123 --- fixed

People

(Reporter: kcochrane, Assigned: kcochrane)

References

Details

(Whiteboard: [fidefe-firefox-view])

Attachments

(1 file)

Recently closed tabs should already be limited by SessionStore via a pref: https://searchfox.org/mozilla-central/source/browser/app/profile/firefox.js#1185

We should remove the limits we've added from the front end to ensure that it's truly driven by that pref alone:
https://searchfox.org/mozilla-central/source/browser/components/firefoxview/recentlyclosed.mjs#32
https://searchfox.org/mozilla-central/source/browser/components/firefoxview/recently-closed-tabs.mjs#41

We'll only need to do it for the new recentlyclosed.mjs since the older module will be going away soon.

The severity field is not set for this bug.
:sfoster, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(sfoster)
Type: defect → task
Priority: -- → P2
Severity: -- → N/A
Flags: needinfo?(sfoster)
Assignee: nobody → kcochrane
Status: NEW → ASSIGNED
Pushed by kcochrane@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/13189fe7391e Remove hardcoded limit of 25 recently closed tabs in Firefox View r=fxview-reviewers,jsudiaman
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: