[meta] GV Content Permissions overhaul meta
Categories
(GeckoView :: General, task, P3)
Tracking
(Not tracked)
People
(Reporter: droeh, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: meta, Whiteboard: [geckoview:m81][geckoview:m82][geckoview:m83][geckoview:m84][geckoview:m85][geckoview:m87][geckoview:m88])
We've decided that it's better to let Gecko handle persisting content permissions, which requires an overhaul of some of PermissionDelegate
, particularly onContentPermissionRequest
. We'll be moving away from callbacks in favor of GeckoResult
s as part of this to bring the API closer to the design of the rest of GV.
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Is this something that is on the roadmap and being worked on? This is still blocking our refactoring work - which itself blocks deprecating and removing some outdated components that are used in Focus.
Reporter | ||
Comment 2•4 years ago
|
||
(In reply to Sebastian Kaspari (:sebastian; :pocmo) from comment #1)
Is this something that is on the roadmap and being worked on? This is still blocking our refactoring work - which itself blocks deprecating and removing some outdated components that are used in Focus.
Yup, I'm working on it at the moment. I'll keep you posted, and we can see about uplifting to 81 once it lands if that would help.
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•3 years ago
|
Reporter | ||
Updated•3 years ago
|
Updated•3 years ago
|
Comment 4•2 years ago
|
||
The bug assignee didn't login in Bugzilla in the last 7 months.
:fluffyemily, could you have a look please?
For more information, please visit auto_nag documentation.
Updated•2 years ago
|
Comment 5•2 years ago
|
||
This landed a while ago and can be closed.
Comment 6•2 years ago
|
||
Agi, do you have a link to the bug where it got landed? I would be interested what exactly has been changed. Thanks.
Comment 7•2 years ago
•
|
||
It's in the dependent bugs :) Before this bug GeckoView would not store permissions and instead delegate them to the app, so it would ask for it every time it was needed. This didn't work well in Gecko where sometimes permissions need to be checked synchronously (and delegating to the app is inherently async) so we moved to a world where GeckoView stores permissions much like Desktop does.
Description
•