From 5917b21702b5006d48fd637ee87d50223a34ce02 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Mon, 4 Jun 2018 12:36:34 +0200 Subject: [PATCH] Remove completely unused `fontScale` property from `PageViewport` The `fontScale` property was added in PR 1531, see commit https://github.com/mozilla/pdf.js/commit/b312719d7e0cf1a45dda0b4ad871a45162ed6f5b in particular, apparently for the sole purpose of supporting the "acroforms" example. However, the `fontScale` property was never used anywhere else in the code-base, and after the modernization of the "acroforms" example in PR 8030 it's been completely unused. Finally, note that there's also a (more suitably named) `scale` property on `PageViewport` instances, which contains the exact same information as the property being removed here. --- src/display/dom_utils.js | 1 - 1 file changed, 1 deletion(-) diff --git a/src/display/dom_utils.js b/src/display/dom_utils.js index ca9b820ea..a6f7ec624 100644 --- a/src/display/dom_utils.js +++ b/src/display/dom_utils.js @@ -212,7 +212,6 @@ var PageViewport = (function PageViewportClosure() { this.width = width; this.height = height; - this.fontScale = scale; } PageViewport.prototype = /** @lends PageViewport.prototype */ { /**