Opened 3 years ago

Closed 3 years ago

#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/
https://codereview.adblockplus.org/29367480/

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.

Change History (5)

comment:1 Changed 3 years ago by oleksandr

  • Description modified (diff)

comment:2 Changed 3 years ago by oleksandr

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed 3 years ago by oleksandr

  • Review URL(s) modified (diff)

comment:4 Changed 3 years ago by abpbot

A commit referencing this issue has landed:
Issue 4721 - Use IndexedDB for storage in Edge

comment:5 Changed 3 years ago by oleksandr

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.