[Pkg-javascript-commits] [pdf.js] 108/116: Only skip the |!isSymbolicFont| check for TrueType (3, 1) cmap tables if no previous cmap table was found (PR 5703 followup)

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository pdf.js.

commit 417800a1b5947b199c4fdb2d5843c45631ee6c40
Author: Jonas Jenwald <jonas.jenwald at gmail.com>
Date:   Thu Feb 19 13:58:03 2015 +0100

    Only skip the |!isSymbolicFont| check for TrueType (3, 1) cmap tables if no previous cmap table was found (PR 5703 followup)
    
    Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=894572.
---
 src/core/fonts.js       |   3 ++-
 test/pdfs/.gitignore    |   1 +
 test/pdfs/bug894572.pdf | Bin 0 -> 41937 bytes
 test/test_manifest.json |   7 +++++++
 4 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/core/fonts.js b/src/core/fonts.js
index 1bcb10f..1f904cd 100644
--- a/src/core/fonts.js
+++ b/src/core/fonts.js
@@ -3184,7 +3184,8 @@ var Font = (function FontClosure() {
             useTable = true;
             // Continue the loop since there still may be a higher priority
             // table.
-          } else if (platformId === 3 && encodingId === 1) {
+          } else if (platformId === 3 && encodingId === 1 &&
+                     (!isSymbolicFont || !potentialTable)) {
             useTable = true;
             if (!isSymbolicFont) {
               canBreak = true;
diff --git a/test/pdfs/.gitignore b/test/pdfs/.gitignore
index 4ebc877..03c7133 100644
--- a/test/pdfs/.gitignore
+++ b/test/pdfs/.gitignore
@@ -29,6 +29,7 @@
 !scan-bad.pdf
 !bug847420.pdf
 !bug860632.pdf
+!bug894572.pdf
 !pdfjsbad1586.pdf
 !freeculture.pdf
 !pdfkit_compressed.pdf
diff --git a/test/pdfs/bug894572.pdf b/test/pdfs/bug894572.pdf
new file mode 100644
index 0000000..77e6714
Binary files /dev/null and b/test/pdfs/bug894572.pdf differ
diff --git a/test/test_manifest.json b/test/test_manifest.json
index b9ba288..9332d52 100644
--- a/test/test_manifest.json
+++ b/test/test_manifest.json
@@ -1106,6 +1106,13 @@
       "lastPage": 1,
       "type": "eq"
     },
+    {  "id": "bug894572",
+       "file": "pdfs/bug894572.pdf",
+       "md5": "e54a6b0451939f685ed37e3d46e16158",
+       "rounds": 1,
+       "link": false,
+       "type": "eq"
+    },
     {  "id": "issue1466",
       "file": "pdfs/issue1466.pdf",
       "md5": "8a8877432e5bb10cfd50d60488d947bb",

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/pdf.js.git