Navigation to relative fragments does not work correctly for OOP pdf.

chrome/browser/resources/pdf/open_pdf_params_parser.js

 // Copyright 2014 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 OpenPDFParamsParser. This parses the open pdf parameters
  * passed in the url to set initial viewport settings for opening the pdf.
  * @param {string} url to be parsed.
@@ skipping 42 matching lines @@
    * See http://www.adobe.com/content/dam/Adobe/en/devnet/acrobat/
    * pdfs/pdf_open_parameters.pdf for details.
    */
   parseOpenPDFParams_: function() {
     var originalUrl = this.url_;
     var paramIndex = originalUrl.search('#');
     if (paramIndex == -1)
       return;
 
     var paramTokens = originalUrl.substring(paramIndex + 1).split('&');
+    // Handle the case of http://foo.com/bar#NAMEDDEST. This is not explicitly
+    // mentioned except by example in the Adobe "PDF Open Parameters" document.

[raymes, 2015/01/13 01:59:31]

How come you removed this?

+    if ((paramTokens.length == 1) && (paramTokens[0].search('=') == -1)) {
+      this.urlParams['nameddest'] = paramTokens[0];

[raymes, 2015/01/11 23:08:41]

We don't actually want to store nameddest in urlParams. Instead, we want to use
it to determine urlParams['page'] based on the list of named destinations which
get sent from the plugin. So instead do something like 

var namedDest;
...
namedDest = paramTokens[0];

and then down the bottom something like:
if ('nameddest' in paramsDictionary) 
  namedDest = paramsDictionary['nameddest'];

and then
if (namedDest) 
  this.urlParams['page'] = namedDestinations_[namedDest];

if (

[Deepak, 2015/01/12 09:21:43]

Done.

[raymes, 2015/01/13 01:59:31]

Please take a look at the comment above again, in particular the last 2 lines of
code. We want to have this right at the bottom of this function

if (namedDest) 
  this.urlParams['page'] = namedDestinations_[namedDest];

+      return;
+    }
+
     var paramsDictionary = {};
     for (var i = 0; i < paramTokens.length; ++i) {
       var keyValueSplit = paramTokens[i].split('=');
       if (keyValueSplit.length != 2)
         continue;
       paramsDictionary[keyValueSplit[0]] = keyValueSplit[1];
     }
 
     if ('page' in paramsDictionary) {
       // |pageNumber| is 1-based, but goToPage() take a zero-based page number.
       var pageNumber = parseInt(paramsDictionary['page']);
       if (!isNaN(pageNumber))
         this.urlParams['page'] = pageNumber - 1;
     }
 
     if ('zoom' in paramsDictionary)
       this.parseZoomParam_(paramsDictionary['zoom']);
   }
+
+    if ('nameddest' in paramsDictionary)
+      this.urlParams['nameddest'] = paramsDictionary['nameddest'];
 };