From e629882f2a43ac251d3767c2cee4c70506385783 Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Tue, 10 Nov 2015 11:13:44 +0100
Subject: [PATCH] Correctly handle /FitH destinations with `null` as the
 parameter

According to http://www.adobe.com/content/dam/Adobe/en/devnet/acrobat/pdfs/PDF32000_2008.pdf#G11.2095870, the vertical position shouldn't change if the parameter is `null`.

Fixes 6615.
---
 web/pdf_viewer.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/web/pdf_viewer.js b/web/pdf_viewer.js
index cec51cf35..0e4f97391 100644
--- a/web/pdf_viewer.js
+++ b/web/pdf_viewer.js
@@ -519,6 +519,12 @@ var PDFViewer = (function pdfViewer() {
         case 'FitBH':
           y = dest[2];
           scale = 'page-width';
+          // According to the PDF spec, section 12.3.2.2, a `null` value in the
+          // parameter should maintain the position relative to the new page.
+          if (y === null && this._location) {
+            x = this._location.left;
+            y = this._location.top;
+          }
           break;
         case 'FitV':
         case 'FitBV':