Refactoring of code by moving navigate_() to separate file.

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 123 matching lines @@
       // We should not change zoom level when we are responsible for initiating
       // the zoom. onZoomChange() is called before setZoomComplete() callback
       // when we initiate the zoom.
       if ((zoomDelta > MIN_ZOOM_DELTA) && !this.setZoomInProgress_)
         this.viewport_.setZoom(zoomChangeInfo.newZoomFactor);
     }.bind(this));
   }
 
   // Parse open pdf parameters.
   this.paramsParser_ = new OpenPDFParamsParser();
+  this.navigator_ = new Navigator();
 }
 
 PDFViewer.prototype = {
   /**
    * @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) {
@@ skipping 186 matching lines @@
    */
   onPasswordSubmitted_: function(event) {
     this.plugin_.postMessage({
       type: 'getPasswordComplete',
       password: event.detail.password
     });
   },
 
   /**
    * @private
-   * Helper function to navigate to given URL. This might involve navigating
-   * within the PDF page or opening a new url (in the same tab or a new tab).
-   * @param {string} url The URL to navigate to.
-   * @param {boolean} newTab Whether to perform the navigation in a new tab or
-   * in the current tab.
-   */
-  navigate_: function(url, newTab) {
-    if (url.length == 0)
-      return;
-    var originalUrl = this.streamDetails_.originalUrl;
-    // If |urlFragment| starts with '#', then it's for the same URL with a
-    // different URL fragment.
-    if (url.charAt(0) == '#') {
-      // if '#' is already present in |originalUrl| then remove old fragment
-      // and add new url fragment.
-      var hashIndex = originalUrl.search('#');
-      if (hashIndex != -1)
-        url = originalUrl.substring(0, hashIndex) + url;
-      else
-        url = originalUrl + url;
-    }
-
-    // If there's no scheme, add http.
-    if (url.indexOf('://') == -1 && url.indexOf('mailto:') == -1)
-      url = 'http://' + url;
-
-    // Make sure url starts with a valid scheme.
-    if (url.indexOf('http://') != 0 &&
-        url.indexOf('https://') != 0 &&
-        url.indexOf('ftp://') != 0 &&
-        url.indexOf('file://') != 0 &&
-        url.indexOf('mailto:') != 0) {
-      return;
-    }
-    // Make sure url is not only a scheme.
-    if (url == 'http://' ||
-        url == 'https://' ||
-        url == 'ftp://' ||
-        url == 'file://' ||
-        url == 'mailto:') {
-      return;
-    }
-
-    if (newTab) {
-      // Prefer the tabs API because it guarantees we can just open a new tab.
-      // window.open doesn't have this guarantee.
-      if (chrome.tabs)
-        chrome.tabs.create({ url: url });
-      else
-        window.open(url);
-    } else {
-      var pageNumber =
-          this.paramsParser_.getViewportFromUrlParams(url).page;
-      if (pageNumber != undefined)
-        this.viewport_.goToPage(pageNumber);
-      else
-        window.location.href = url;
-    }
-  },
-
-  /**
-   * @private
    * An event handler for handling message events received from the plugin.
    * @param {MessageObject} message a message event.
    */
   handlePluginMessage_: function(message) {
     switch (message.data.type.toString()) {
       case 'documentDimensions':
         this.documentDimensions_ = message.data;
         this.viewport_.setDocumentDimensions(this.documentDimensions_);
         // If we received the document dimensions, the password was good so we
         // can dismiss the password screen.
@@ skipping 24 matching lines @@
         this.sendScriptingMessage_(message.data);
         break;
       case 'goToPage':
         this.viewport_.goToPage(message.data.page);
         break;
       case 'loadProgress':
         this.updateProgress_(message.data.progress);
         break;
       case 'navigate':
         // If in print preview, always open a new tab.
-        if (this.isPrintPreview_)
+        if (this.isPrintPreview_) {
           this.navigate_(message.data.url, true);
-        else
-          this.navigate_(message.data.url, message.data.newTab);
+        } else {
+          this.navigator_.navigate(message.data.url, message.data.newTab,
+          this.streamDetails_.originalUrl, this.viewport_, this.paramsParser_);

[raymes, 2015/01/21 07:03:44]

Can you please pass in the original URL, viewport and paramsParser on
construction. They never change so we might as well pass them in on construction

[Deepak, 2015/01/21 08:33:41]

Done.

+        }
         break;
       case 'setNamedDestinations':
         this.paramsParser_.namedDestinations = message.data.namedDestinations;
         break;
       case 'setScrollPosition':
         var position = this.viewport_.position;
         if (message.data.x !== undefined)
           position.x = message.data.x;
         if (message.data.y !== undefined)
           position.y = message.data.y;
@@ skipping 214 matching lines @@
               this.streamDetails_.tabId != -1);
   },
 
   /**
    * @type {Viewport} the viewport of the PDF viewer.
    */
   get viewport() {
     return this.viewport_;
   }
 };