Browse Source

Don't map glyphs to Unicode position 0x0E33, i.e. Thai character SARA AM (bug1046314)

*A similar approach as in PR 5705.*

Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1046314.

According to https://dxr.mozilla.org/mozilla-central/source/gfx/harfbuzz/src/hb-ot-shape-complex-thai.cc#270-365, `0x0E33` is treated as a special case (by the font shaping code in Firefox). Hence it seems reasonable to skip it when adjusting the font mapping.
Jonas Jenwald 10 years ago
parent
commit
2b1a13ba28
  1. 1
      src/core/fonts.js
  2. 1
      test/pdfs/.gitignore
  3. BIN
      test/pdfs/bug1046314.pdf
  4. 7
      test/test_manifest.json

1
src/core/fonts.js

@ -2698,6 +2698,7 @@ var Font = (function FontClosure() { @@ -2698,6 +2698,7 @@ var Font = (function FontClosure() {
fontCharCode === 0x7F || // Control char
fontCharCode === 0xAD || // Soft hyphen
fontCharCode === 0xA0 || // Non breaking space
fontCharCode === 0x0E33 || // Thai character SARA AM
fontCharCode === 0x25CC || // Dotted circle (combining mark)
(fontCharCode >= 0x80 && fontCharCode <= 0x9F) || // Control chars
// Prevent drawing characters in the specials unicode block.

1
test/pdfs/.gitignore vendored

@ -86,6 +86,7 @@ @@ -86,6 +86,7 @@
!bug866395.pdf
!bug1027533.pdf
!bug1028735.pdf
!bug1046314.pdf
!bug1065245.pdf
!basicapi.pdf
!mixedfonts.pdf

BIN
test/pdfs/bug1046314.pdf

Binary file not shown.

7
test/test_manifest.json

@ -557,6 +557,13 @@ @@ -557,6 +557,13 @@
"rounds": 1,
"type": "load"
},
{ "id": "bug1046314",
"file": "pdfs/bug1046314.pdf",
"md5": "fc658439f44cd2dd27c8bee7e7a8344e",
"rounds": 1,
"link": false,
"type": "eq"
},
{ "id": "hudsonsurvey",
"file": "pdfs/hudsonsurvey.pdf",
"md5": "bf0e6576a7b6c2fe7485bce1b78e006f",

Loading…
Cancel
Save