Merge pull request #12339 from Snuffleupagus/bug-1663390

Support broken /FitH destinations that are missing the "top" value (bug 1663390)
mozilla · Sep 7, 2020 · ed47f77 · ed47f77
2 parents 20c8915 + f78b348
commit ed47f77
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/web/base_viewer.js b/web/base_viewer.js
@@ -852,6 +852,10 @@ class BaseViewer {
         if (y === null && this._location) {
           x = this._location.left;
           y = this._location.top;
+        } else if (typeof y !== "number") {
+          // The "top" value isn't optional, according to the spec, however some
+          // bad PDF generators will pretend that it is (fixes bug 1663390).
+          y = pageHeight;
         }
         break;
       case "FitV":