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

Move updating of this._maxUid into PDFHistory._updateInternalState

Browse Source 
There's no compelling reason to update this property *manually* in multiple places, since that's error-prone with any future code changes, given that `_updateInternalState` is always called just before anyway.
This commit is contained in:
Jonas Jenwald 2020-10-12 12:26:19 +02:00
parent 295716f496
commit b302fd3a6e
1 changed files with 1 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
web/pdf_history.js
View File

@ -143,9 +143,6 @@ class PDFHistory {
state.uid,
/* removeTemporary = */ true
);
if (this._uid > this._maxUid) {
this._maxUid = this._uid;
}
if (destination.rotation !== undefined) {
this._initialRotation = destination.rotation;
@ -410,7 +407,6 @@ class PDFHistory {
if (shouldReplace) {
window.history.replaceState(newState, "", newUrl);
} else {
this._maxUid = this._uid;
window.history.pushState(newState, "", newUrl);
}
@ -534,6 +530,7 @@ class PDFHistory {
}
this._destination = destination;
this._uid = uid;
this._maxUid = Math.max(this._maxUid, uid);
// This should always be reset when `this._destination` is updated.
this._numPositionUpdates = 0;
}
@ -688,9 +685,6 @@ class PDFHistory {
state.uid,
/* removeTemporary = */ true
);
if (this._uid > this._maxUid) {
this._maxUid = this._uid;
}
if (isValidRotation(destination.rotation)) {
this.linkService.rotation = destination.rotation;