Closed
Bug 1315927
Opened 8 years ago
Closed 8 years ago
ignore firstPartyDomain and userContextId in PermissionStatus
Categories
(Core :: Security, defect, P2)
Core
Security
Tracking
()
RESOLVED
FIXED
mozilla52
| Tracking | Status | |
|---|---|---|
| firefox52 | --- | fixed |
People
(Reporter: allstars.chh, Assigned: allstars.chh)
References
(Blocks 1 open bug)
Details
(Whiteboard: [tor][domsecurity-active])
Attachments
(1 file, 1 obsolete file)
|
11.80 KB,
patch
|
allstars.chh
:
review+
|
Details | Diff | Splinter Review |
In bug 1302046 we ignore userContextId and firstPartyDomain in nsPermission and nsPermissionManager. We could also do the same thing for PermissionStatus.
|
Assignee | |
Comment 1•8 years ago
|
||
The test is copy from dom/permission/tests/test_permissions_api.html, I add privacy.firstparty.isolation on it.
Attachment #8808560 -
Flags: review?(amarchesini)
|
||
Comment 2•8 years ago
|
||
Comment on attachment 8808560 [details] [diff] [review]
Patch.
Review of attachment 8808560 [details] [diff] [review]:
-----------------------------------------------------------------
::: browser/components/originattributes/test/mochitest/mochitest.ini
@@ +2,5 @@
> +support-files =
> + file_empty.html
> +
> +[test_permissions_api.html]
> +
extra line
Attachment #8808560 -
Flags: review?(amarchesini) → review+
|
|
Assignee | |
Comment 3•8 years ago
|
||
removed extra line
Attachment #8808560 -
Attachment is obsolete: true
Attachment #8808893 -
Flags: review+
Pushed by yhuang@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/804bcd698ce5
ignore firstPartyDomain and userContextId in PermissionStatus. r=baku
|
||
Comment 5•8 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
|
||
Updated•8 years ago
|
Priority: -- → P2
You need to log in
before you can comment on or make changes to this bug.
Description
•