Enable Storage API for GeckoView
Categories
(Core :: Storage: Quota Manager, enhancement, P3)
Tracking
()
People
(Reporter: overholt, Assigned: fluffyemily, NeedInfo)
References
Details
(Whiteboard: [geckoview:p3])
Can we enable persist() and estimate() in GeckoView without knowing how apps that use GeckoView will expose associated UI?
Maybe we don't necessarily want user consent, but instead some API that allows apps to specify the amount of storage available? They could of course set that based on user input if they wish. We'd also need some stuff to allow apps to inspect the storage used for each origin and clear that, etc.
I don't really think it's too useful to prompt for each site, but I'd welcome input from the UX folks on that.
Reporter | ||
Comment 2•6 years ago
|
||
There are a variety of APIs available from the QuotaManager that we could potentially expose via custom GV APIs. Agreed that it probably needs UX input.
Hmm, I'm not sure who we can NI from UX. Anthony, maybe you can help?
Comment 4•6 years ago
|
||
Hey James, maybe Shorlander since he's been working on Fenix? He has most of the context here.
Assignee | ||
Comment 5•6 years ago
|
||
This is related to the work I've been looking at for https://bugzilla.mozilla.org/show_bug.cgi?id=1527074, which is essentially wiring up persist()
and ensuring it asks for Android and user permissioning.
Updated•6 years ago
|
Updated•6 years ago
|
Comment 6•6 years ago
|
||
Emily is looking at storage permissions bug 1527074. Priority P3 because this feature is not needed for Fenix MVP.
The test runner app will need to accept storage requests.
Assignee | ||
Comment 7•6 years ago
|
||
Comment 8•6 years ago
|
||
(In reply to Emily Toop (:fluffyemily) from comment #7)
^ Just to be clear, those bug fixes were for Storage API test failures, not enabling the Storage API.
Assignee | ||
Comment 9•6 years ago
|
||
So, bug 1527074 enabled the Storage API and bug 1547100 enabled the tests that ensure that persist
and estimate
work.
Assignee | ||
Updated•5 years ago
|
Description
•