PDF: Add shortcut to toggle fit-to-{page,width}.

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';
 
 /**
  * @return {number} Width of a scrollbar in pixels
  */
 function getScrollbarWidth() {
@@ skipping 378 matching lines @@
       case 71: // g key.
         if (this.toolbar_ && (e.ctrlKey || e.metaKey) && e.altKey) {
           this.toolbarManager_.showToolbars();
           this.toolbar_.selectPageNumber();
         }
         return;
       case 219:  // left bracket.
         if (e.ctrlKey)
           this.rotateCounterClockwise_();
         return;
+      case 220:  // Backslash key.
+        if (e.ctrlKey) {
+          if (this.viewport_.fittingType == Viewport.FittingType.FIT_TO_PAGE) {
+            this.fitToWidth_();
+          } else {
+            this.fitToPage_();
+          }
+          this.zoomToolbar_.toggleFitButtonUIState();

[raymes, 2016/09/05 03:21:40]

Could we just call fitToggle on the zoom toolbar? I know it will be calling back
out to here, but maybe it makes the state tracking more simple? WDYT?

[Lei Zhang, 2016/09/07 00:39:09]

Done.

+        }
+        return;
       case 221:  // right bracket.
         if (e.ctrlKey)
           this.rotateClockwise_();
         return;
     }
 
     // Give print preview a chance to handle the key event.
     if (!fromScriptingAPI && this.isPrintPreview_) {
       this.sendScriptingMessage_({
         type: 'sendKeyEvent',
@@ skipping 26 matching lines @@
   /**
    * @private
    * Rotate the plugin counter-clockwise.
    */
   rotateCounterClockwise_: function() {
     this.plugin_.postMessage({
       type: 'rotateCounterclockwise'
     });
   },
 
+  /**
+   * @private
+   * Set zoom to "fit to page".
+   */
   fitToPage_: function() {
     this.viewport_.fitToPage();
     this.toolbarManager_.forceHideTopToolbar();
   },
 
   /**
    * @private
+   * Set zoom to "fit to width".
+   */
+  fitToWidth_: function() {
+    this.viewport_.fitToWidth();
+    this.toolbarManager_.forceHideTopToolbar();
+  },
+
+  /**
+   * @private
    * Notify the plugin to print.
    */
   print_: function() {
     this.plugin_.postMessage({
       type: 'print'
     });
   },
 
   /**
    * @private
@@ skipping 418 matching lines @@
    * Each bookmark is an Object containing a:
    * - title
    * - page (optional)
    * - array of children (themselves bookmarks)
    * @type {Array} the top-level bookmarks of the PDF.
    */
   get bookmarks() {
     return this.bookmarks_;
   }
 };