[WORK_IN_PROGRESS] PDF::Save() fix for out-of-process-pdf.

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="open_pdf_params_parser.js">
 <include src="pdf_scripting_api.js">
 <include src="viewport.js">
@@ skipping 17 matching lines @@
 /**
  * The minimum number of pixels to offset the toolbar by from the bottom and
  * right side of the screen.
  */
 PDFViewer.MIN_TOOLBAR_OFFSET = 15;
 
 /**
  * Creates a new PDFViewer. There should only be one of these objects per
  * document.
  */
-function PDFViewer() {
+function PDFViewer(streamDetails) {
+  this.streamDetails = streamDetails;
   this.loaded = false;
 
   // The sizer element is placed behind the plugin element to cause scrollbars
   // to be displayed in the window. It is sized according to the document size
   // of the pdf and zoom level.
   this.sizer_ = $('sizer');
   this.toolbar_ = $('toolbar');
   this.pageIndicator_ = $('page-indicator');
   this.progressBar_ = $('progress-bar');
   this.passwordScreen_ = $('password-screen');
@@ skipping 21 matching lines @@
   this.plugin_.addEventListener('message', this.handlePluginMessage_.bind(this),
                                 false);
 
   // Handle scripting messages from outside the extension that wish to interact
   // with it. We also send a message indicating that extension has loaded and
   // is ready to receive messages.
   window.addEventListener('message', this.handleScriptingMessage_.bind(this),
                           false);
   this.sendScriptingMessage_({type: 'readyToReceive'});
 
-  // If the viewer is started from a MIME type request, there will be a
-  // background page and stream details object with the details of the request.
-  // Otherwise, we take the query string of the URL to indicate the URL of the
-  // PDF to load. This is used for print preview in particular.
-  if (chrome.extension.getBackgroundPage &&
-      chrome.extension.getBackgroundPage()) {
-    this.streamDetails =
-        chrome.extension.getBackgroundPage().popStreamDetails();
-  }
-
-  if (!this.streamDetails) {
-    // The URL of this page will be of the form
-    // "chrome-extension://<extension id>?<pdf url>". We pull out the <pdf url>
-    // part here.
-    var url = window.location.search.substring(1);
-    this.streamDetails = {
-      streamUrl: url,
-      originalUrl: url,
-      responseHeaders: ''
-    };
-  }
-
   this.plugin_.setAttribute('src', this.streamDetails.originalUrl);
   this.plugin_.setAttribute('stream-url', this.streamDetails.streamUrl);
   var headers = '';
   for (var header in this.streamDetails.responseHeaders) {
     headers += header + ': ' +
         this.streamDetails.responseHeaders[header] + '\n';
   }
   this.plugin_.setAttribute('headers', headers);
 
   if (window.top == window)
@@ skipping 11 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-link').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({mode: 'manual', scope: 'per-tab'},
                                 this.afterZoom_.bind(this));
     chrome.tabs.onZoomChange.addListener(function(zoomChangeInfo) {
@@ skipping 29 matching lines @@
     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;
+        this.viewport_.position = position;
       }
-    };
+    }.bind(this);
     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;
+        this.viewport_.position = position;
       }
-    };
+    }.bind(this);
 
     switch (e.keyCode) {
       case 32:  // Space key.
         if (e.shiftKey)
           pageUpHandler();
         else
           pageDownHandler();
         return;
       case 33:  // Page up key.
         pageUpHandler();
@@ skipping 28 matching lines @@
           position.x += Viewport.SCROLL_INCREMENT;
           this.viewport.position = position;
         }
         return;
       case 40:  // Down arrow key.
         if (fromScriptingAPI) {
           position.y += Viewport.SCROLL_INCREMENT;
           this.viewport.position = position;
         }
         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;
     }
   },
 
   /**
    * @private
    * 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 281 matching lines @@
   },
 
   /**
    * @type {Viewport} the viewport of the PDF viewer.
    */
   get viewport() {
     return this.viewport_;
   }
 };
 
-var viewer = new PDFViewer();
+var viewer;
+function initPlugin(streamDetails) {
+  viewer = new PDFViewer(streamDetails);
+}
+
+function initUrl() {
+  // If the viewer is started from the browser plugin, the plugin ID will be
+  // passed in.
+  var params = window.location.search.substring(1).split('=');
+  var viewId = '';
+  if (params.length == 2 && params[0] == 'id')
+    viewId = params[1];
+
+  if (viewId) {
+    chrome.runtime.sendMessage('mhjfbmdgcfjbbpaeojofohoefgiehjai',
+                               {viewId: viewId},
+                               initPlugin);
+    return;
+  }
+
+  // The viewer may be started via a background page from the extension.
+  var streamDetails;
+  if (chrome.extension &&
+      chrome.extension.getBackgroundPage &&
+      chrome.extension.getBackgroundPage()) {
+    streamDetails =
+        chrome.extension.getBackgroundPage().popStreamDetails();
+    if (streamDetails) {
+      initPlugin(streamDetails);
+      return;
+    }
+  }
+
+  // The viewer may be started directly by passing in the URL of the PDF to load
+  // as the query string. This is used for print preview in particular. The URL
+  // of this page will be of the form
+  // "chrome-extension://<extension id>?<pdf url>". We pull out the <pdf url>
+  // part here.
+  var url = window.location.search.substring(1);
+  streamDetails = {
+    streamUrl: url,
+    originalUrl: url,
+    responseHeaders: ''
+  };
+  initPlugin(streamDetails);
+}
+
+initUrl();