Considering removing third-party "persistent-storage" prompting support
Categories
(Core :: Storage: localStorage & sessionStorage, enhancement, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox72 | --- | fixed |
People
(Reporter: annevk, Assigned: tnguyen)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
This came up during bug 1572461. It's problematic that a third-party can prompt for persistent storage as this leads to UX that's not as understandable as it could be and once third-party isolation is a thing we do by default this would be even more problematic.
As we did already for notifications, I think we should stop allowing third-party contexts to prompt the user. If the third-party has the permission through a first-party visit it seems fine if they continue to keep access to it (provided there's no isolation).
|
||
Comment 1•5 years ago
|
||
I agree :)
|
Reporter | |
Updated•5 years ago
|
|
||
Updated•5 years ago
|
|
|
Reporter | |
Comment 2•5 years ago
|
||
Marking this as blocking bug 1572461 as per Johann the infrastructure will/should change such that this effectively cannot work anymore.
|
|
Reporter | |
Updated•5 years ago
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 3•5 years ago
|
||
I'd wait for landing bug 1483631 first (I added a new API in that bug)
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
||
Comment 6•5 years ago
|
||
Backed out changeset dde0354521ab (bug 1583142) for causing PermissionDelegateHandler.cpp bustages
push that caused the backout: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&resultStatus=testfailed%2Cbusted%2Cexception%2Cusercancel%2Crunnable&revision=dde0354521ab5a77cedfbaceb885c538bbfa4176
backout: https://hg.mozilla.org/integration/autoland/rev/6d6dbb3b7d2fe556f6328874702977058a05b11d
|
||
Comment 8•5 years ago
|
||
| bugherder | ||
|
|
Assignee | |
Updated•5 years ago
|
Description
•