Closed Bug 1631560 Opened 4 years ago Closed 4 years ago

Stop getting topLevelAntiTrackingPrincipal/GetTopLevelStorageAreaPrincipal in ContentBlocking::ShouldAllowAccessFor().

Categories

(Core :: Privacy: Anti-Tracking, task)

task

Tracking

()

RESOLVED FIXED
mozilla77
Tracking Status
firefox77 --- fixed

People

(Reporter: timhuang, Assigned: timhuang)

References

Details

Attachments

(1 file)

We use GetParentPrincipalAndTrackingOrigin() to get the topLevelAntiTrackingPrincipal/GetTopLevelStorageAreaPrincipal. However, all its callers don't really need the top-level principal. So, we can remove the part of getting the top-level principal from this function.

Status: NEW → ASSIGNED

We don't really need the
topLevelAntiTrackingPrincipal/GetTopLevelStorageAreaPrincipal in
ContentBlocking::ShouldAllowAccessFor(). So, we can remove the code that
gets the topLevelAntiTrackingPrincipal/GetTopLevelStorageAreaPrincipal.

Pushed by tihuang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e1676ab58d34
Stop getting topLevelAntiTrackingPrincipal/GetTopLevelStorageAreaPrincipal in ContentBlocking::ShouldAllowAccessFor(). r=dimi,baku
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: