OOP PDF: Change the save toolbar button to match the save menu behavior.

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 88 matching lines @@
 
   // Setup the button event listeners.
   $('fit-to-width-button').addEventListener('click',
       this.viewport_.fitToWidth.bind(this.viewport_));
   $('fit-to-page-button').addEventListener('click',
       this.viewport_.fitToPage.bind(this.viewport_));
   $('zoom-in-button').addEventListener('click',
       this.viewport_.zoomIn.bind(this.viewport_));
   $('zoom-out-button').addEventListener('click',
       this.viewport_.zoomOut.bind(this.viewport_));
-  $('save-button-link').href = this.streamDetails.originalUrl;
+  $('save-button').addEventListener('click', this.save_.bind(this));
   $('print-button').addEventListener('click', this.print_.bind(this));
 
   // Setup the keyboard event listener.
   document.onkeydown = this.handleKeyEvent_.bind(this);
 
   // Set up the zoom API.
   if (chrome.tabs) {
     chrome.tabs.setZoomSettings(this.streamDetails.tabId,
                                 {mode: 'manual', scope: 'per-tab'},
                                 this.afterZoom_.bind(this));
@@ skipping 101 matching lines @@
           this.viewport.position = position;
         }
         return;
       case 65:  // a key.
         if (e.ctrlKey || e.metaKey) {
           this.plugin_.postMessage({
             type: 'selectAll',
           });
         }
         return;
-      case 83:  // s key.
-        if (e.ctrlKey || e.metaKey) {
-          // Simulate a click on the button so that the <a download ...>
-          // attribute is used.
-          $('save-button-link').click();
-          // Since we do the saving of the page.
-          e.preventDefault();
-        }
-        return;
       case 80:  // p key.
         if (e.ctrlKey || e.metaKey) {
           this.print_();
           // Since we do the printing of the page.
           e.preventDefault();
         }
         return;
       case 219:  // left bracket.
         if (e.ctrlKey) {
           this.plugin_.postMessage({
@@ skipping 16 matching lines @@
    * Notify the plugin to print.
    */
   print_: function() {
     this.plugin_.postMessage({
       type: 'print',
     });
   },
 
   /**
    * @private
+   * Notify the plugin to save.
+   */
+  save_: function() {
+    this.plugin_.postMessage({
+      type: 'save',
+    });
+  },
+
+  /**
+   * @private
    * Handle open pdf parameters. This function updates the viewport as per
    * the parameters mentioned in the url while opening pdf. The order is
    * important as later actions can override the effects of previous actions.
    */
   handleURLParams_: function() {
     if (this.urlParams_.page)
       this.viewport_.goToPage(this.urlParams_.page);
     if (this.urlParams_.position) {
       // Make sure we don't cancel effect of page parameter.
       this.viewport_.position = {
@@ skipping 287 matching lines @@
     window.parent.postMessage(message, '*');
   },
 
   /**
    * @type {Viewport} the viewport of the PDF viewer.
    */
   get viewport() {
     return this.viewport_;
   }
 };