Merge pull request mozilla#861 from notmasteryet/issue-840

vingtetun · vingtetun · commit c0291e9fa3db · 2011-12-02T05:20:24.000-08:00
(mozilla#840) fix font stretching and overlapping characters encoding
diff --git a/src/canvas.js b/src/canvas.js
@@ -575,7 +575,7 @@ var CanvasGraphics = (function canvasGraphics() {
       ctx.scale(1, -1);
       ctx.translate(current.x, -1 * current.y);
       ctx.transform.apply(ctx, fontMatrix);
-      ctx.scale(1 / textHScale, 1);
+      ctx.scale(textHScale, 1);
     },
     getTextGeometry: function canvasGetTextGeometry() {
       var geometry = {};
@@ -651,7 +651,7 @@ var CanvasGraphics = (function canvasGraphics() {
         ctx.transform.apply(ctx, current.textMatrix);
         ctx.translate(current.x, current.y);
 
-        ctx.scale(1 / textHScale, 1);
+        ctx.scale(textHScale, 1);
         for (var i = 0; i < glyphsLength; ++i) {
 
           var glyph = glyphs[i];
diff --git a/src/fonts.js b/src/fonts.js
@@ -1705,10 +1705,18 @@ var Font = (function Font() {
 
         var cidToGidMap = properties.cidToGidMap || [];
         var gidToCidMap = [0];
-        for (var j = cidToGidMap.length - 1; j >= 0; j--) {
-          var gid = cidToGidMap[j];
-          if (gid)
-            gidToCidMap[gid] = j;
+        if (cidToGidMap.length > 0) {
+          for (var j = cidToGidMap.length - 1; j >= 0; j--) {
+            var gid = cidToGidMap[j];
+            if (gid)
+              gidToCidMap[gid] = j;
+          }
+          // filling the gaps using CID above the CIDs currently used in font
+          var nextCid = cidToGidMap.length;
+          for (var i = 1; i < numGlyphs; i++) {
+            if (!gidToCidMap[i])
+              gidToCidMap[i] = nextCid++;
+          }
         }
 
         var glyphs = [], ids = [];
diff --git a/test/pdfs/.gitignore b/test/pdfs/.gitignore
@@ -16,3 +16,4 @@
 !alphatrans.pdf
 !devicen.pdf
 !cmykjpeg.pdf
+!issue840.pdf
diff --git a/test/pdfs/issue840.pdf b/test/pdfs/issue840.pdf
diff --git a/test/test_manifest.json b/test/test_manifest.json
@@ -290,5 +290,11 @@
        "link": true,
        "rounds": 1,
        "type": "eq"
+    },
+    {  "id": "issue840",
+       "file": "pdfs/issue840.pdf",
+       "md5": "20d88011dd7e3c4fb5274979094dab93",
+       "rounds": 1,
+       "type": "eq"
     }
 ]

Original file line number	Diff line number	Diff line change
`@@ -290,5 +290,11 @@`
`290`	`290`	`"link": true,`
`291`	`291`	`"rounds": 1,`
`292`	`292`	`"type": "eq"`
	`293`	`+ },`
	`294`	`+ { "id": "issue840",`
	`295`	`+ "file": "pdfs/issue840.pdf",`
	`296`	`+ "md5": "20d88011dd7e3c4fb5274979094dab93",`
	`297`	`+ "rounds": 1,`
	`298`	`+ "type": "eq"`
`293`	`299`	`}`
`294`	`300`	`]`