Closed Bug 1654546 Opened 4 years ago Closed 4 years ago

No shield icon in the url bar when the top-level page is sandboxed

Categories

(Firefox :: Protections UI, defect, P2)

defect

Tracking

()

VERIFIED FIXED
85 Branch
Tracking Status
firefox85 --- verified

People

(Reporter: dimi, Assigned: pbz)

References

()

Details

Attachments

(3 files, 1 obsolete file)

Details can be found in Bug 1647681 Comment 0.

Group: mozilla-employee-confidential
Severity: -- → S3
Component: Privacy: Anti-Tracking → Protections UI
Priority: -- → P2
Product: Core → Firefox
Severity: S3 → S2

Seems like the protection panel works fine with a nullprincipal, except for the TP toggle. It's currently disabled by the check here:
https://searchfox.org/mozilla-central/rev/f081504642a115cb8236bea4d8250e5cb0f39b02/toolkit/components/antitracking/ContentBlockingAllowList.jsm#49 (isContentPrincipal).
Disabling / hiding the toggle for the top level sandbox case would work, but then the user loses the ability to turn TP off. I'll look into how we can persist the trackingprotection permission for the top level sandbox case.

Assignee: nobody → pbz
Status: NEW → ASSIGNED
Attachment #9189964 - Attachment is obsolete: true
Pushed by pzuhlcke@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e7c2aad1aade
Don't set contentBlockingAllowListPrincipal to NullPrincipal for sandboxed documents. r=timhuang,emilio
https://hg.mozilla.org/integration/autoland/rev/0796dd620879
Test that we show the protection UI for top level sandboxed pages. r=timhuang
https://hg.mozilla.org/integration/autoland/rev/101d2f54673f
Added test for the contentBlockingAllowListPrincipal. r=timhuang
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
Flags: qe-verify+

I have managed to reproduce this bug using the test page from bug 1647681#c0, the shield icon was not displayed on an affected Nightly build (79.0a1, 20200623213840).

The issue is verified as fixed on latest Beta 85.0b4 (20201220193140) under macOS 10.15, Windows 10 x64 and Ubuntu 18.04 x64.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: