Closed Bug 1878636 Opened 1 year ago Closed 1 year ago

Remove filterPrivateTabs as extensions can't create private tabs in non-private windows anymore

Categories

(Firefox :: Session Restore, task, P3)

Desktop
All
task

Tracking

()

RESOLVED FIXED
124 Branch
Tracking Status
firefox124 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

The concern for users' privacy in bug 961380 is obviously admirable, but it isn't possible anymore for extensions to create arbitrary private tabs in non-private windows, so now https://searchfox.org/mozilla-central/rev/26dd94a024f21971e539a8650c2f177323c98fe2/toolkit/modules/sessionstore/PrivacyFilter.sys.mjs#121-143 is just a busy-loop that doesn't serve any purpose. Code coverage shows that the inner lines that remove tabs are never executed, which makes sense because the case the loop is worried about (private tabs in non-private windows) can't happen.

We should remove this code.

Severity: -- → N/A
Type: defect → task
Priority: -- → P3
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Blocks: 1740261
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/519454a6fe47 remove filterPrivateTabs as it creates needless performance churn, r=sfoster,sessionstore-reviewers,kaya,farre
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 124 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: