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

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

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.