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.
Pdf.js doesn't store so much data, but it's always worth to get rid of a synchronous storage method.
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
You need to log in before you can comment on or make changes to this bug.