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

Merge pull request #9356 from Snuffleupagus/PDFHistory-maxUid-pageHide-fixes

Browse Source 
Two small `PDFHistory` fixes, concerning the `this._maxUid` property and the 'pagehide' event
This commit is contained in:
Tim van der Meij 2018-01-12 22:07:15 +01:00 committed by GitHub
commit ba0a3aebd0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
web/pdf_history.js
View File

@ -133,6 +133,9 @@ class PDFHistory {
let destination = state.destination;
this._updateInternalState(destination, state.uid,
/* removeTemporary = */ true);
if (this._uid > this._maxUid) {
this._maxUid = this._uid;
}
if (destination.rotation !== undefined) {
this.initialRotation = destination.rotation;
@ -510,6 +513,9 @@ class PDFHistory {
let destination = state.destination;
this._updateInternalState(destination, state.uid,
/* removeTemporary = */ true);
if (this._uid > this._maxUid) {
this._maxUid = this._uid;
}
if (isValidRotation(destination.rotation)) {
this.linkService.rotation = destination.rotation;
@ -541,10 +547,10 @@ class PDFHistory {
_boundEvents.pageHide = (evt) => {
// Attempt to push the `this._position` into the browser history when
// navigating away from the document. This is *only* done if the history
// is currently empty, since otherwise an existing browser history entry
// is empty/temporary, since otherwise an existing browser history entry
// will end up being overwritten (given that new entries cannot be pushed
// into the browser history when the 'unload' event has already fired).
if (!this._destination) {
if (!this._destination || this._destination.temporary) {
this._tryPushCurrentPosition();
}
};