Closed Bug 1200098 Opened 4 years ago Closed 4 years ago

update graphite2 library to upstream release 1.3.2

Categories

(Core :: Graphics: Text, defect)

43 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: martin_hosken, Assigned: jfkthame)

References

Details

(Whiteboard: gfx-noted)

Attachments

(3 files)

This release of graphite2 replaces 1.3.0. It fixes all the fuzz bugs you and we have found and resolves some bidi issues.
Tryserver says we hit a (bidi-related) crash in one of our reftests:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8f5e1475fd83

So we'll need to resolve that before taking an update here.
when you call gr_make_seg try oring the rtl parameter with 2. Since your strings are all monodirectional, no point in running full bidi on them. Having said that, there may well be a bug since in theory whether you run full bidi or not on a monodirectional string shouldn't make any difference.
(In reply to martin_hosken from comment #2)
> when you call gr_make_seg try oring the rtl parameter with 2. Since your
> strings are all monodirectional, no point in running full bidi on them.
> Having said that, there may well be a bug since in theory whether you run
> full bidi or not on a monodirectional string shouldn't make any difference.

I tried this but it still hits the crash.
Note that the crash occurs on a (diacritic-only) segment that we're shaping with dir=ltr (because diacritics inherit their direction from the base character), while aSilf->dir() is rtl. So it still ends up calling seg->reverseSlots().
Whiteboard: gfx-noted
Summary: graphite 1.3.1 released → graphite2 1.3.1 released
Untested fix in repo (well it doesn't crash in normal cases, but not tested against your test case).

In addition. The copyright message that had to be added for 1.3.0 has been removed and all files have the same license and copyright. So that patch can be removed.
Release 1.3.2 is now out; we should go ahead and update. (This is primarily about bug-fixing and robustness, following an intensive fuzz-testing period.)
Summary: graphite2 1.3.1 released → update graphite2 library to upstream release 1.3.2
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Upstream has modified their code such that it no longer carries this additional copyright, so we can remove it from about:license per comment 5 above.
Attachment #8658671 - Flags: review?(gerv)
This is just a minor tweak for efficiency; we ought to have been doing this all along. There should be no effect on behavior.
Attachment #8658673 - Flags: review?(jdaggett)
Comment on attachment 8658671 [details] [diff] [review]
patch 2 - Remove the data-shrinker copyright notice that was added for graphite 1.3.0; no longer present in release 1.3.2

Review of attachment 8658671 [details] [diff] [review]:
-----------------------------------------------------------------

r=gerv; thanks for coming back and cleaning up :-)

Gerv
Attachment #8658671 - Flags: review?(gerv) → review+
Attachment #8658673 - Flags: review?(jdaggett) → review+
Attachment #8658670 - Flags: review?(jdaggett) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/5e3c7c4b60893bda2d2eae7fcd2aa2bab64c04c8
Bug 1200098 - Update graphite2 library to release 1.3.2 from upstream. r=jdaggett

https://hg.mozilla.org/integration/mozilla-inbound/rev/66e7e47e3144da359a8d0e0e4ebfa4d12d33840a
Bug 1200098 - patch 2 - Remove the data-shrinker copyright notice that was added for graphite 1.3.0; no longer present in release 1.3.2. r=gerv

https://hg.mozilla.org/integration/mozilla-inbound/rev/ca6aca7378307f47d5807f4d40ad5f27cd67babf
Bug 1200098 - patch 3 - Pass the gr_nobidi flag when shaping with graphite2, as we split text into unidirectional runs ahead of time. r=jdaggett
Blocks: 1193757
Blocks: 1193999
Blocks: 1194002
Blocks: 1194006
Depends on: 1223002
You need to log in before you can comment on or make changes to this bug.