Stop using localStorage in pdf.js

RESOLVED FIXED

Status

()

RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: marco, Unassigned)

Tracking

17 Branch
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [pdfjs-c-performance])

(Reporter)

Description

6 years ago
localStorage has many performance problems, IndexedDB would be better.
This was already filed upstream as https://github.com/mozilla/pdf.js/issues/1046, seems like we should just reopen that.
(Reporter)

Comment 2

6 years ago
Pdf.js doesn't store so much data, but it's always worth to get rid of a synchronous storage method.

Updated

6 years ago
Whiteboard: [pdfjs-c-performance]
(Reporter)

Comment 4

6 years ago
I see in browser/extensions/pdfjs/ we're using Firefox preferences to store pdf.js settings, and we aren't using localStorage at all.
Maybe the only thing we could do is removing the function localStorageEnabledTest and the variable isLocalStorageEnabled, that are now unused.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Updated

5 years ago
See Also: → bug 865893
You need to log in before you can comment on or make changes to this bug.