Open
Bug 1488636
Opened 7 years ago
Updated 3 years ago
Add a test for the triggering principal fallback case
Categories
(Core :: Privacy: Anti-Tracking, enhancement)
Core
Privacy: Anti-Tracking
Tracking
()
NEW
People
(Reporter: ehsan.akhgari, Unassigned)
References
(Blocks 1 open bug)
Details
Our coverage data shows that we currently don't cover triggering principal fallback case in the channel version of IsFirstPartyStorageAccessGrantedFor(): https://codecov.io/gh/mozilla/gecko-dev/src/e126996/toolkit/components/antitracking/AntiTrackingCommon.cpp#L426
I'm not sure how to construct a test case that would capture this. baku, do you have any ideas? It would be nice to have a test to cover this.
Flags: needinfo?(amarchesini)
|
||
Comment 1•7 years ago
|
||
Actually I don't think there is a channel unable to return a null result principal:
https://searchfox.org/mozilla-central/source/caps/nsScriptSecurityManager.cpp#244-254
I'll think about it. Keeping the NI.
|
||
Updated•7 years ago
|
Priority: -- → P3
|
||
Updated•3 years ago
|
Severity: normal → S3
|
||
Updated•3 years ago
|
Flags: needinfo?(amarchesini)
|
|
||
Updated•3 years ago
|
Component: Protections UI → Privacy: Anti-Tracking
Product: Firefox → Core
|
|
||
Updated•3 years ago
|
Severity: S3 → --
Priority: P3 → --
You need to log in
before you can comment on or make changes to this bug.
Description
•