Closed Bug 1467108 Opened 3 years ago Closed 3 years ago

Update harfbuzz to 1.8.1


(Core :: Graphics: Text, defect)

Not set



Tracking Status
firefox63 --- fixed


(Reporter: ionnv, Assigned: RyanVM)




(1 file)

Overview of changes leading to 1.8.0
Tuesday, June 5, 2018
- Update to Unicode 11.0.0.
Depends on: 1443812
My understanding is that this is a pretty massive update. What's the urgency on this from our end, Jonathan?
Ever confirmed: true
Flags: needinfo?(jfkthame)
Worth pointing out also that version 1.7.7 was also simultaneously released, which gives a bit better idea of the volume of changes.

Overview of changes leading to 1.7.7
Tuesday, June 5, 2018
- Lots of internal changes, but not yet exposed externally.
- All HarfBuzz objects are significantly smaller in size now.
- Sinhala: Position repha on top of post-consonant, not base.
  This better matches Windows 10 behavior, which was changed
  from previous Windows versions.
- New build options:
  o New cpp macro HB_NO_ATEXIT
  o New cpp macro HB_SUBSET_BUILTIN
- Significant libharfbuzz-subset changes. API subject to change.
- New API in libharfbuzz:


A hashmap implementation:
I've got a patch which builds, but layout/reftests/font-features/subsuper-fallback.html doesn't appear to be very happy about it.
Hmm, that'll need investigating. I suspect something about hb_set has changed/broken.

In any case, I was going to suggest waiting until the new cycle; I don't think there's any reason to rush this in.
Flags: needinfo?(jfkthame)
This was indeed a bug in harfbuzz... actually there was a pre-existing bug, but minor internal changes happened to make it easier to hit, and one of our testcases stumbled across it. Filed as, and now fixed upstream.

Re-trying your patch plus the upstream fix:
And now 1.8.1 is released. :)
Summary: Update harfbuzz to 1.8.0 → Update harfbuzz to 1.8.1
I'll revisit this bug after the Gecko 63 version bump with whatever the current version is by then.
Comment on attachment 8987948 [details] [diff] [review]
update harfbuzz to version 1.8.1

Review of attachment 8987948 [details] [diff] [review]:

::: gfx/harfbuzz/README-mozilla
@@ +10,5 @@
> +Note that harfbuzz.pc, harfbuzz-icu.pc, and harfbuzz-subset.pc are not present
> +in the upstream Git repository. They are created at build time by the HarfBuzz
> +build system. Since we don't use that build system in Mozilla, it is
> +necessary to refresh these files when updating HarfBuzz and check them into the
> +Mozilla tree.

Do we actually need the generated *.pc files for anything? (I don't think so... in which case we should be able to forget this mess and just import from the upstream release directly into our tree, now that src/hb-version.h is provided from upstream.)
Comment on attachment 8987948 [details] [diff] [review]
update harfbuzz to version 1.8.1

Review of attachment 8987948 [details] [diff] [review]:

rs=me;  feel free to simply drop the .pc files as discussed.
Attachment #8987948 - Flags: review?(jfkthame) → review+
Pushed by
Update HarfBuzz to version 1.8.1. r=jfkthame
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Blocks: 1473064
You need to log in before you can comment on or make changes to this bug.