Closed Bug 1484788 Opened 2 years ago Closed 2 years ago

Make the Disable Protection button in the control centre UI work for third-party cookie blocking settings


(Firefox :: Protections UI, enhancement, P3)




Firefox 63


(Reporter: ehsan, Assigned: ehsan)



(Keywords: meta)

My plan for this bug is as follows:

 * Port PrivateBrowsingTrackingProtectionWhitelist to use session permissions.  This has the advantage that the permissions will be synch'ed up with content processes without us having to write custom IPC code for that.  It will also allow us to simplify the code that queries the permission (and make it slightly more efficient by removing the need to run JS there!).  Session permissions aren't written to disk so this is a suitable choice from a private browsing perspective.
 * Refactor the code in nsChannelClassifier <> responsible for checking these settings to make it usable elsewhere.
 * Use that same code in places where we perform our storage checks.
Ah, that would be nice, I already asked why this wasn't initially done a while ago and I don't think I really got a conclusive answer.
Depends on: 1484868
Depends on: 1484876
Priority: -- → P3
Depends on: 1486185
Depends on: 1486926
This bug can be closed when all of its dependencies are fixed, there will be no work done in this bug itself.
Keywords: meta
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Depends on: 1491061
You need to log in before you can comment on or make changes to this bug.