commit 7593306a8770d5fd7d884532a6dd012df55b4d7f
author raymes <raymes@chromium.org> Wed Feb 25 01:05:07 2015
committer Commit bot <commit-bot@chromium.org> Wed Feb 25 01:05:27 2015
tree 995ad4d7048150ed3449b35ccbdfda056885b501
parent 259948539c7d01c968c6fb18f459b9adb19f4f51

Fix left/right keyboard scrolling in PDFs

Non-existent dictionary members were trying to be accessed.

BUG=461587

Review URL: https://codereview.chromium.org/959443003

Cr-Commit-Position: refs/heads/master@{#317935}

chrome/browser/resources/pdf/pdf.js

diff --git a/chrome/browser/resources/pdf/pdf.js b/chrome/browser/resources/pdf/pdf.js
index 3046b5b..23d9bbed 100644
--- a/chrome/browser/resources/pdf/pdf.js
+++ b/chrome/browser/resources/pdf/pdf.js
@@ -262,7 +262,7 @@
       case 37:  // Left arrow key.
         if (!(e.altKey || e.ctrlKey || e.metaKey || e.shiftKey)) {
           // Go to the previous page if there are no horizontal scrollbars.
-          if (!this.viewport_.documentHasScrollbars().x) {
+          if (!this.viewport_.documentHasScrollbars().horizontal) {
             this.viewport_.goToPage(this.viewport_.getMostVisiblePage() - 1);
             // Since we do the movement of the page.
             e.preventDefault();
@@ -281,7 +281,7 @@
       case 39:  // Right arrow key.
         if (!(e.altKey || e.ctrlKey || e.metaKey || e.shiftKey)) {
           // Go to the next page if there are no horizontal scrollbars.
-          if (!this.viewport_.documentHasScrollbars().x) {
+          if (!this.viewport_.documentHasScrollbars().horizontal) {
             this.viewport_.goToPage(this.viewport_.getMostVisiblePage() + 1);
             // Since we do the movement of the page.
             e.preventDefault();