Opened on 12/09/2016 at 06:36:02 AM
Closed on 01/12/2017 at 10:08:29 AM
#4721 closed change (fixed)
Use IndexedDB for storage in Edge
| Reported by: | oleksandr | Assignee: | |
|---|---|---|---|
| Priority: | Unknown | Milestone: | |
| Module: | Platform | Keywords: | |
| Cc: | sebastian, kzar | Blocked By: | |
| Blocking: | Platform: | Edge | |
| Ready: | no | Confidential: | no |
| Tester: | Unknown | Verified working: | no |
| Review URL(s): |
https://codereview.adblockplus.org/29367151/ |
||
Description (last modified by oleksandr)
Background
In #4023 we have already implemented a storage in IndexedDB for Edge. Since this is the only thing that is keeping us from releasing a development build from our branch, move to IndexedDB in our bookmark on adblockpluschrome as well.
We should probably make it easily reversible, so that in future we could merge our bookmark easier onto the master, so it would be best to simply overload ext.storage for Edge builds.
What to change
Move storage from browser.storage.local to IndexedDB for Edge by overloading ext.storage.
Attachments (0)
Change History (5)
comment:2 Changed on 12/09/2016 at 06:40:36 AM by oleksandr
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:4 Changed on 01/12/2017 at 10:06:21 AM by abpbot
comment:5 Changed on 01/12/2017 at 10:08:29 AM by oleksandr
- Resolution set to fixed
- Status changed from reviewing to closed
Note: See
TracTickets for help on using
tickets.
A commit referencing this issue has landed:
Issue 4721 - Use IndexedDB for storage in Edge