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

Drop third parameter to replaceState/pushState

Browse Source 
This was introduced in #3582 to work around https://crbug.com/274024 .
The bug in Chrome has been fixed a long time ago (at least 33), so let's
simplify the code.
This commit is contained in:
Rob Wu 2018-01-26 12:50:25 +01:00
parent 628e70fbb5
commit e904b8a67c
1 changed files with 2 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
web/pdf_history.js
View File

@ -281,23 +281,10 @@ class PDFHistory {
this._updateInternalState(destination, newState.uid);
if (shouldReplace) {
if (typeof PDFJSDev !== 'undefined' &&
PDFJSDev.test('FIREFOX || MOZCENTRAL')) {
// Providing the third argument causes a SecurityError for file:// URLs.
window.history.replaceState(newState, '');
} else {
window.history.replaceState(newState, '', document.URL);
}
window.history.replaceState(newState, '');
} else {
this._maxUid = this._uid;
if (typeof PDFJSDev !== 'undefined' &&
PDFJSDev.test('FIREFOX || MOZCENTRAL')) {
// Providing the third argument causes a SecurityError for file:// URLs.
window.history.pushState(newState, '');
} else {
window.history.pushState(newState, '', document.URL);
}
window.history.pushState(newState, '');
}
if (typeof PDFJSDev !== 'undefined' && PDFJSDev.test('CHROME') &&