Thai pages not displaying some characters correctly

VERIFIED FIXED in Firefox 6

Status

()

Core
Layout: Text
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Cww, Assigned: jfkthame)

Tracking

6 Branch
mozilla8
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(firefox6+ fixed, firefox7+ fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
From input: http://input.mozilla.com/en-US/?product=firefox&sentiment=sad&date_end=&date_start=2011-07-12&version=6.0&q=thai

Thai characters aren't working correctly or consistently.

For example, some of the text on http://www.glo.or.th/index.php works but others don't. http://www.bts.co.th/th/index.asp is another good example.

It seems that most people are complaining about thai facebook but that could be a function of the site's popularity.  All reports are OSX.

Updated

6 years ago
Component: General → Layout: Text
Product: Firefox → Core
QA Contact: general → layout.fonts-and-text

Comment 1

6 years ago
FWIW, the font info says that the corrupted display is diplayed in the "Microsoft Sans Serif" font.

Which matches the CSS style, font-family: "MS Sans Serif","Microsoft Sans Serif";
(Assignee)

Comment 2

6 years ago
The problem here is that Microsoft Sans Serif includes Thai characters, but doesn't have AAT tables. I believe this causes Core Text to decide that it can't actually be used, and substitute some other font during its shaping. The result is that we end up with glyph IDs that don't correspond to the font that thebes thinks it's using.
(Assignee)

Comment 3

6 years ago
Created attachment 547392 [details] [diff] [review]
patch, prefer HarfBuzz to Core Text for shaping Thai on OS X

This fixes the problem by preferring the HB shaper, which doesn't switch fonts behind our back.

It is configured to apply on OS X only for now, so that we don't change behavior on Windows; I believe the MS shaping engines do "sequence checking" and insert dotted circles in cases that they consider ill-formed. While this is a debatable practice, I don't want to change it as a side-effect of this bug; we should look into it separately.
Assignee: nobody → jfkthame
Attachment #547392 - Flags: review?(jdaggett)
(Reporter)

Updated

6 years ago
tracking-firefox6: --- → ?
(Reporter)

Updated

6 years ago
tracking-firefox7: --- → ?

Updated

6 years ago
Attachment #547392 - Flags: review?(jdaggett) → review+
(Assignee)

Comment 4

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/f83df39873c7
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/f83df39873c7
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
(Reporter)

Comment 6

6 years ago
Comment on attachment 547392 [details] [diff] [review]
patch, prefer HarfBuzz to Core Text for shaping Thai on OS X

I know it doesn't have bake time on trunk yet but this looks pretty safe and I really really want this on beta ASAP.
Attachment #547392 - Flags: approval-mozilla-beta?
Attachment #547392 - Flags: approval-mozilla-aurora?

Comment 7

6 years ago
Comment on attachment 547392 [details] [diff] [review]
patch, prefer HarfBuzz to Core Text for shaping Thai on OS X

Approved for releases/mozilla-aurora and releases/mozilla-beta. Please land asap.
Attachment #547392 - Flags: approval-mozilla-beta?
Attachment #547392 - Flags: approval-mozilla-beta+
Attachment #547392 - Flags: approval-mozilla-aurora?
Attachment #547392 - Flags: approval-mozilla-aurora+
(Assignee)

Comment 8

6 years ago
http://hg.mozilla.org/releases/mozilla-aurora/rev/9b3f60d9e8a7
http://hg.mozilla.org/releases/mozilla-beta/rev/f3e82fad65b2
status-firefox6: --- → fixed
status-firefox7: --- → fixed

Comment 9

6 years ago
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:7.0a2) Gecko/20110731 Firefox/7.0a2
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:6.0) Gecko/20100101 Firefox/6.0

Verified issue on FF 6.0b4 and Aurora using the links provided as examples in the Description - characters are correctly displayed.

Setting resolution to VERIFIED FIXED.
Status: RESOLVED → VERIFIED
tracking-firefox6: ? → +
tracking-firefox7: ? → +
You need to log in before you can comment on or make changes to this bug.