Merge pull request #5127 from Hengjie/fix-pdfdocument-null

Fix issue where pdfDocument is null
mozilla · Aug 4, 2014 · 87de3cd · 87de3cd
2 parents 3fc746c + eaa5877
commit 87de3cd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/web/secondary_toolbar.js b/web/secondary_toolbar.js
@@ -101,7 +101,9 @@ var SecondaryToolbar = {
   },
 
   lastPageClick: function secondaryToolbarLastPageClick(evt) {
-    PDFView.page = PDFView.pdfDocument.numPages;
+    if (PDFView.pdfDocument) {
+      PDFView.page = PDFView.pdfDocument.numPages;
+    }
     this.close();
   },
 

diff --git a/web/viewer.js b/web/viewer.js
@@ -2376,8 +2376,8 @@ window.addEventListener('keydown', function keydown(evt) {
         }
         break;
       case 35: // end
-        if (PresentationMode.active ||
-            PDFView.page < PDFView.pdfDocument.numPages) {
+        if (PresentationMode.active || (PDFView.pdfDocument &&
+            PDFView.page < PDFView.pdfDocument.numPages)) {
           PDFView.page = PDFView.pdfDocument.numPages;
           handled = true;
         }