Opened 4 years ago

Closed 3 years ago

#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.

Change History (5)

comment:1 Changed 4 years ago by sergz

  • Cc sergz added

comment:2 Changed 3 years ago by sebastian

  • Cc scottlow added

comment:3 Changed 3 years ago by sebastian

  • Keywords edge removed
  • Platform changed from Unknown / Cross platform to Edge

comment:4 Changed 3 years ago by sebastian

  • Sensitive unset

comment:5 Changed 3 years ago by oleksandr

  • Resolution set to rejected
  • Status changed from new to closed

We have decided to not split the files, but use a different storage mechanism instead (IndexedDB). Closing this ticket.

Note: See TracTickets for help on using tickets.