Add an MRU cache for CheckContentBlockingAllowList()

RESOLVED FIXED in Firefox 67

Status

()

enhancement
RESOLVED FIXED
4 months ago
4 months ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
mozilla67
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox67 fixed)

Details

(Whiteboard: [anti-tracking])

Attachments

(2 attachments)

Assignee

Description

4 months ago

This funtion tends to get called repeatedly for the same window and channel arguments, so it makes sense to add a small MRU cache to save the cost of recomputing the answer when we had just computed it moments ago.

Assignee

Updated

4 months ago
Blocks: 1516540
Assignee

Updated

4 months ago
No longer blocks: 1516540
Whiteboard: [anti-tracking]
Target Milestone: --- → mozilla67
Assignee

Updated

4 months ago
Blocks: 1516540

Comment 3

4 months ago
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b9d970f20f18
Part 1: Add an API for registering a callback to get called when something changes in the environment which can change the result of anti-tracking checks; r=baku
https://hg.mozilla.org/integration/autoland/rev/02e125fcde8a
Part 2: Add an MRU cache for CheckContentBlockingAllowList(); r=baku

Comment 4

4 months ago
bugherder
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.