Closed Bug 793282 Opened 12 years ago Closed 11 years ago

Stop using localStorage in pdf.js

Categories

(Firefox :: PDF Viewer, defect)

Product:
Firefox
Component:
PDF Viewer
Version:
17 Branch
Platform:
x86_64
Windows 7
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: marco, Unassigned)

References

Details

(Whiteboard: [pdfjs-c-performance]) 

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.
Whiteboard: [pdfjs-c-performance]
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
Closed: 11 years ago
Resolution: --- → FIXED
See Also: → 865893
You need to log in before you can comment on or make changes to this bug.