Opened on 03/02/2016 at 12:19:37 PM
Closed on 12/09/2016 at 06:29:09 AM
#3716 closed change (rejected)
Split up files stored in storage.local
Reported by: | sebastian | Assignee: | |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Platform | Keywords: | |
Cc: | sergz, scottlow | Blocked By: | |
Blocking: | Platform: | Edge | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Background
We currently store all filter data as one value in storage.local. However, values are currently limited to 1MB (and 5MB total) on Edge, causing Adblock Plus to re-initialized on each startup, as with EasyList alone that quota is already exceeded and storage.local.set() silently fails.
What to change
When storing files in storage.local in lib/io.js split files up into multiple key-value pairs.
Attachments (0)
Change History (5)
comment:1 Changed on 04/04/2016 at 02:49:32 PM by sergz
- Cc sergz added
comment:2 Changed on 05/07/2016 at 01:32:41 AM by sebastian
- Cc scottlow added
comment:3 Changed on 05/23/2016 at 10:36:25 AM by sebastian
- Keywords edge removed
- Platform changed from Unknown / Cross platform to Edge
comment:4 Changed on 06/10/2016 at 11:27:49 PM by sebastian
- Sensitive unset
comment:5 Changed on 12/09/2016 at 06:29:09 AM by oleksandr
- Resolution set to rejected
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.
We have decided to not split the files, but use a different storage mechanism instead (IndexedDB). Closing this ticket.