OOP PDF - Add OpenPDFParamsParser class

chrome/browser/resources/pdf/pdf_open_params_parser.js

Index: chrome/browser/resources/pdf/pdf_open_params_parser.js
diff --git a/chrome/browser/resources/pdf/pdf_open_params_parser.js b/chrome/browser/resources/pdf/pdf_open_params_parser.js
new file mode 100644
index 0000000000000000000000000000000000000000..c333f700d11bc75bd3a41ecad99d7d3cc3fbd752
--- /dev/null
+++ b/chrome/browser/resources/pdf/pdf_open_params_parser.js
@@ -0,0 +1,55 @@
+// 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.
+ */
+function OpenPDFParamsParser(url) {
+  this.url_ = url;
+  this.viewportSettings_ = {};
+}
+
+OpenPDFParamsParser.prototype = {
+  /**
+   * @private
+   * Parse open PDF parameters. These parameters are mentioned in the url
+   * and specify actions to be performed when opening pdf files.
+   * 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('&');
+    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.viewportSettings_['page'] = pageNumber - 1;
+    }
+  },
+
+  /**
+   * @type {ViewportSettings} the viewport settings to be used for opening pdf.
+   */
+  get viewportSettings() {
+    this.parseOpenPDFParams_();

[raymes, 2014/08/27 01:02:24]

Why not just do this on construction? Then you wouldn't need a getter here, you
could just expose the viewportSettings member directly.

[Nikhil, 2014/08/27 10:16:09]

Done.

+    return this.viewportSettings_;
+  }
+};