Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #8794 from Snuffleupagus/ViewHistory-rm-migration-code

Browse Source 
Remove the migration code for old localStorage data in `ViewHistory`
This commit is contained in:
Tim van der Meij 2017-08-19 15:09:17 +02:00 committed by GitHub
commit 7969802739
1 changed files with 1 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
web/view_history.js
View File

@ -73,25 +73,7 @@ class ViewHistory {
PDFJSDev.test('FIREFOX || MOZCENTRAL')) { PDFJSDev.test('FIREFOX || MOZCENTRAL')) {
resolve(sessionStorage.getItem('pdfjs.history')); resolve(sessionStorage.getItem('pdfjs.history'));
} else { } else {
let value = localStorage.getItem('pdfjs.history'); resolve(localStorage.getItem('pdfjs.history'));
// TODO: Remove this key-name conversion after a suitable time-frame.
// Note that we only remove the old 'database' entry if it looks like
// it was created by PDF.js, to avoid removing someone else's data.
if (!value) {
let databaseStr = localStorage.getItem('database');
if (databaseStr) {
try {
let database = JSON.parse(databaseStr);
if (typeof database.files[0].fingerprint === 'string') {
localStorage.setItem('pdfjs.history', databaseStr);
localStorage.removeItem('database');
value = databaseStr;
}
} catch (ex) { }
}
}
resolve(value);
} }
}); });
} }