Closed Bug 745780 Opened 8 years ago Closed 8 years ago

update harfbuzz to latest code from upstream

Categories

(Core :: Graphics: Text, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla14

People

(Reporter: jfkthame, Assigned: jfkthame)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

It'd be good to take a harfbuzz update now that Arabic fallback (presentation-form) shaping is implemented; this will be particularly beneficial on Android devices that are shipping with OpenType-less versions of Droid Sans Arabic.
Blocks: 676068
Assignee: nobody → jfkthame
Attachment #615402 - Flags: review?(jdaggett)
Note that the testcase involving a diacritic within the lam-alef ligature will currently fail (marked accordingly in the manifest), as Behdad hasn't yet implemented support for that. I expect that'll come in a minor update before long, but would like to get the basic code landed ASAP so that we can get it into the Android product; the use of diacritics within ligatures is a rare enough edge case that it's not critical if we don't have that right away.
Attachment #615403 - Flags: review?(jdaggett)
Comment on attachment 615403 [details] [diff] [review]
reftests for arabic presentation-form shaping

> +fails HTTP(..) != arabic-fallback-4.html arabic-fallback-4-notref.html

Why the failure here?
(In reply to John Daggett (:jtd) from comment #3)

> Why the failure here?

Ah that's the lam-alef ligature.  Let's comment that out for now with a comment noting what it's testing.
I'm seeing a couple reftest failures on try under XP:
https://tbpl.mozilla.org/php/getParsedLog.php?id=11061026&tree=Try&full=1

bidi/bidi-006.html
bidi/bidi-006-j.html

The usual suspects...
Oh yes - that's bug 727736 again. As that isn't yet addressed in upstream harfbuzz, we need to re-apply our patch locally. I'll attach it here for reference.
Comment on attachment 615403 [details] [diff] [review]
reftests for arabic presentation-form shaping

r+ with failing test commented out for now
Attachment #615403 - Flags: review?(jdaggett) → review+
Attachment #616894 - Flags: review?(jdaggett) → review+
Comment on attachment 615402 [details] [diff] [review]
patch, harfbuzz code from upstream

We're going to need reftests for the Indic shaping but we don't need those now since Indic shaping isn't on by default.
Attachment #615402 - Flags: review?(jdaggett) → review+
(In reply to John Daggett (:jtd) from comment #9)
> Comment on attachment 615402 [details] [diff] [review]
> patch, harfbuzz code from upstream
> 
> We're going to need reftests for the Indic shaping but we don't need those
> now since Indic shaping isn't on by default.

Right - it's known to be incomplete as yet (but Behdad has a test suite and is working on the failing issues).
Comment on attachment 615402 [details] [diff] [review]
patch, harfbuzz code from upstream

Requesting approval to land for mozilla-14. This *does* affect fennec, as we use harfbuzz for text shaping; the motivation for landing it now is that this update will fix the currently-completely-broken rendering of Arabic script on certain devices (e.g. LG Optimus, see bug 676068). Without this, the fennec product is unusable for Arabic-script readers with such devices.
Attachment #615402 - Flags: approval-mozilla-central?
Comment on attachment 616894 [details] [diff] [review]
patch from bug 727736 rebased on top of the harfbuzz update

This is just re-applying the patch from bug 727736 - needed along with the harfbuzz update so that we don't regress rendering of old Hebrew/Arabic fonts that lack proper diacritic support.
Attachment #616894 - Flags: approval-mozilla-central?
Attachment #615402 - Flags: approval-mozilla-central? → approval-mozilla-central+
Attachment #616894 - Flags: approval-mozilla-central? → approval-mozilla-central+
Depends on: 747834
You need to log in before you can comment on or make changes to this bug.