Fix for PDFs with lots of named destinations take a long time to load.

chrome/browser/resources/pdf/navigator.js

 // Copyright 2015 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';
 
 /**
  * Creates a new Navigator for navigating to links inside or outside the PDF.
  * @param {string} originalUrl The original page URL.
  * @param {Object} viewport The viewport info of the page.
@@ skipping 53 matching lines @@
     }
     // Make sure inputURL is not only a scheme.
     if (url == 'http://' ||
         url == 'https://' ||
         url == 'ftp://' ||
         url == 'file://' ||
         url == 'mailto:') {
       return;
     }
 
-    if (newTab) {
+    if (newTab)
       this.navigateInNewTabCallback_(url);
-    } else {
-      var pageNumber =
-          this.paramsParser_.getViewportFromUrlParams(url).page;
-      if (pageNumber != undefined)
-        this.viewport_.goToPage(pageNumber);
-      else
-        this.navigateInCurrentTabCallback_(url);
-    }
+    else
+      this.onViewportReceived_(url);

[raymes, 2015/02/16 22:44:04]

This isn't what I meant. Please keep this the same as what you had before, but
instead of having an inline function ( function(initialViewport)... please pull
make it out into a separate function

[Deepak, 2015/02/17 07:12:12]

Done.

+  },
+
+   /**
+   * @private
+   * Function to decide weather to scroll the page to corresponding named
+   * destination or open the url in the current tab.
+   * param {string} url The new url for name destination or opening it in
+   * current tab.
+   */
+  onViewportReceived_: function(url) {
+    this.paramsParser_.getViewportFromUrlParams(
+        url, function(viewportPosition) {
+        var pageNumber = viewportPosition.page;
+        if (pageNumber != undefined)
+          this.viewport_.goToPage(pageNumber);
+        else
+          this.navigateInCurrentTabCallback_(url);
+    }.bind(this));
   }
 };