Finish StoragePermissionRequest

Bug #1408109 reported by Chris Coulson on 2015-01-06
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Oxide
Low
Chris Coulson

Bug Description

Currently WebContext.storageAccessPermissionDelegate can be used to override per-access cookie permissions. The intention was that it would eventually be used for all storage types, but that work was never finished.

The feature also lacks any unit tests

Changed in oxide:
importance: Undecided → Low
status: New → Triaged
Chris Coulson (chrisccoulson) wrote :

Given that this feature has no tests, has never been completed and AFAICT is not used by any application, I wonder whether we should just remove this API (but keeping a dummy WebContext.storageAccessPermissionDelegate property). I don't want to leave a half-completed feature around, I don't think it's worth spending time on finishing this, and it's going to make the implementation of site-settings more complicated (as well as being superseded by that)

Changed in oxide:
assignee: nobody → Chris Coulson (chrisccoulson)
milestone: none → branch-1.12
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Related blueprints