PDF: Handle the space bar key like page down.

chrome/browser/resources/pdf/pdf.js

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 'use strict';
 
 <include src="../../../../ui/webui/resources/js/util.js">
 <include src="pdf_scripting_api.js">
 <include src="viewport.js">
 
@@ skipping 146 matching lines @@
    * @private
    * Handle key events. These may come from the user directly or via the
    * scripting API.
    * @param {KeyboardEvent} e the event to handle.
    */
   handleKeyEvent_: function(e) {
     var position = this.viewport_.position;
     // Certain scroll events may be sent from outside of the extension.
     var fromScriptingAPI = e.type == 'scriptingKeypress';
 
+    var pageUpHandler = function() {
+      // Go to the previous page if we are fit-to-page.
+      if (this.viewport_.fittingType == Viewport.FittingType.FIT_TO_PAGE) {
+        this.viewport_.goToPage(this.viewport_.getMostVisiblePage() - 1);
+        // Since we do the movement of the page.
+        e.preventDefault();
+      } else if (fromScriptingAPI) {
+        position.y -= this.viewport.size.height;
+        this.viewport.position = position;
+      }
+    };
+    var pageDownHandler = function() {
+      // Go to the next page if we are fit-to-page.
+      if (this.viewport_.fittingType == Viewport.FittingType.FIT_TO_PAGE) {
+        this.viewport_.goToPage(this.viewport_.getMostVisiblePage() + 1);
+        // Since we do the movement of the page.
+        e.preventDefault();
+      } else if (fromScriptingAPI) {
+        position.y += this.viewport.size.height;
+        this.viewport.position = position;
+      }
+    };
+
     switch (e.keyCode) {
+      case 32:  // Space key.
+        if (e.shiftKey)
+          pageUpHandler();
+        else
+          pageDownHandler();
+        return;
       case 33:  // Page up key.
-        // Go to the previous page if we are fit-to-page.
-        if (this.viewport_.fittingType == Viewport.FittingType.FIT_TO_PAGE) {
-          this.viewport_.goToPage(this.viewport_.getMostVisiblePage() - 1);
-          // Since we do the movement of the page.
-          e.preventDefault();
-        } else if (fromScriptingAPI) {
-          position.y -= this.viewport.size.height;
-          this.viewport.position = position;
-        }
+        pageUpHandler();
         return;
       case 34:  // Page down key.
-        // Go to the next page if we are fit-to-page.
-        if (this.viewport_.fittingType == Viewport.FittingType.FIT_TO_PAGE) {
-          this.viewport_.goToPage(this.viewport_.getMostVisiblePage() + 1);
-          // Since we do the movement of the page.
-          e.preventDefault();
-        } else if (fromScriptingAPI) {
-          position.y += this.viewport.size.height;
-          this.viewport.position = position;
-        }
+        pageDownHandler();
         return;
       case 37:  // Left arrow key.
         // Go to the previous page if there are no horizontal scrollbars.
         if (!this.viewport_.documentHasScrollbars().x) {
           this.viewport_.goToPage(this.viewport_.getMostVisiblePage() - 1);
           // Since we do the movement of the page.
           e.preventDefault();
         } else if (fromScriptingAPI) {
           position.x -= Viewport.SCROLL_INCREMENT;
           this.viewport.position = position;
@@ skipping 327 matching lines @@
 
   /**
    * @type {Viewport} the viewport of the PDF viewer.
    */
   get viewport() {
     return this.viewport_;
   }
 };
 
 var viewer = new PDFViewer();