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

chrome/test/data/pdf/params_parser_test.js

Index: chrome/test/data/pdf/params_parser_test.js
diff --git a/chrome/test/data/pdf/params_parser_test.js b/chrome/test/data/pdf/params_parser_test.js
index 34bccf59f57302197b71015dc1b44143dd97b293..00d717103650907f6dd0911d1b30469d3b429e0d 100644
--- a/chrome/test/data/pdf/params_parser_test.js
+++ b/chrome/test/data/pdf/params_parser_test.js
@@ -16,57 +16,67 @@ var tests = [
     var url = "http://xyz.pdf";
 
     // Checking #nameddest.
-    var urlParams = paramsParser.getViewportFromUrlParams(url + "#RU");
-    chrome.test.assertEq(urlParams.page, 26);
+    paramsParser.getViewportFromUrlParams(url + "#RU", function(urlParams) {

[raymes, 2015/02/16 02:02:31]

nit: here and below urlParams->viewportPosition

[Deepak, 2015/02/16 06:45:38]

Done.

+      chrome.test.assertEq(urlParams.page, 26);
+    });
 
     // Checking #nameddest=name.
-    urlParams = paramsParser.getViewportFromUrlParams(url + "#nameddest=US");
-    chrome.test.assertEq(urlParams.page, 0);
+    paramsParser.getViewportFromUrlParams(
+        url + "#nameddest=US", function(urlParams) {
+          chrome.test.assertEq(urlParams.page, 0);
+    });
 
     // Checking #page=pagenum nameddest.The document first page has a pagenum
     // value of 1.
-    urlParams = paramsParser.getViewportFromUrlParams(url + "#page=6");
-    chrome.test.assertEq(urlParams.page, 5);
+    paramsParser.getViewportFromUrlParams(url + "#page=6", function(urlParams) {
+      chrome.test.assertEq(urlParams.page, 5);
+    });
 
     // Checking #zoom=scale.
-    urlParams = paramsParser.getViewportFromUrlParams(url + "#zoom=200");
-    chrome.test.assertEq(urlParams.zoom, 2);
+    paramsParser.getViewportFromUrlParams(
+        url + "#zoom=200", function(urlParams) {
+          chrome.test.assertEq(urlParams.zoom, 2);
+    });
 
     // Checking #zoom=scale,left,top.
-    urlParams = paramsParser.getViewportFromUrlParams(url +
-        "#zoom=200,100,200");
-    chrome.test.assertEq(urlParams.zoom, 2);
-    chrome.test.assertEq(urlParams.position.x, 100);
-    chrome.test.assertEq(urlParams.position.y, 200);
+    paramsParser.getViewportFromUrlParams(
+        url + "#zoom=200,100,200", function(urlParams) {
+          chrome.test.assertEq(urlParams.zoom, 2);
+          chrome.test.assertEq(urlParams.position.x, 100);
+          chrome.test.assertEq(urlParams.position.y, 200);
+    });
 
     // Checking #nameddest=name and zoom=scale.
-    urlParams = paramsParser.getViewportFromUrlParams(url +
-        "#nameddest=UY&zoom=150");
-    chrome.test.assertEq(urlParams.page, 22);
-    chrome.test.assertEq(urlParams.zoom, 1.5);
+    paramsParser.getViewportFromUrlParams(
+        url + "#nameddest=UY&zoom=150", function(urlParams) {
+          chrome.test.assertEq(urlParams.page, 22);
+          chrome.test.assertEq(urlParams.zoom, 1.5);
+    });
 
     // Checking #page=pagenum and zoom=scale.
-    urlParams = paramsParser.getViewportFromUrlParams(url +
-        "#page=2&zoom=250");
-    chrome.test.assertEq(urlParams.page, 1);
-    chrome.test.assertEq(urlParams.zoom, 2.5);
+    paramsParser.getViewportFromUrlParams(
+        url + "#page=2&zoom=250", function(urlParams) {
+          chrome.test.assertEq(urlParams.page, 1);
+          chrome.test.assertEq(urlParams.zoom, 2.5);
+    });
 
     // Checking #nameddest=name and zoom=scale,left,top.
-    urlParams = paramsParser.getViewportFromUrlParams(url +
-        "#nameddest=UY&zoom=150,100,200");
-    chrome.test.assertEq(urlParams.page, 22);
-    chrome.test.assertEq(urlParams.zoom, 1.5);
-    chrome.test.assertEq(urlParams.position.x, 100);
-    chrome.test.assertEq(urlParams.position.y, 200);
+    paramsParser.getViewportFromUrlParams(
+        url + "#nameddest=UY&zoom=150,100,200", function(urlParams) {
+          chrome.test.assertEq(urlParams.page, 22);
+          chrome.test.assertEq(urlParams.zoom, 1.5);
+          chrome.test.assertEq(urlParams.position.x, 100);
+          chrome.test.assertEq(urlParams.position.y, 200);
+    });
 
     // Checking #page=pagenum and zoom=scale,left,top.
-    urlParams = paramsParser.getViewportFromUrlParams(url +
-        "#page=2&zoom=250,100,200");
-    chrome.test.assertEq(urlParams.page, 1);
-    chrome.test.assertEq(urlParams.zoom, 2.5);
-    chrome.test.assertEq(urlParams.position.x, 100);
-    chrome.test.assertEq(urlParams.position.y, 200);
-
+    paramsParser.getViewportFromUrlParams(
+        url + "#page=2&zoom=250,100,200", function(urlParams) {
+          chrome.test.assertEq(urlParams.page, 1);
+          chrome.test.assertEq(urlParams.zoom, 2.5);
+          chrome.test.assertEq(urlParams.position.x, 100);
+          chrome.test.assertEq(urlParams.position.y, 200);
+    });
     chrome.test.succeed();
   }
 ];