1358502 - Update harfbuzz to version 1.4.6 [was: IPA Characters Overlap When Entering Text on 64-bit Windows systems]

Status: VERIFIED FIXED

(Core :: Graphics: Text, defect, P3)

Description

[Timea Zsoldos [:zstimi/tzsoldos], Desktop QA]

Follow up for bug 1279925, to track the remaining issues for 64-bit systems, (Windows 10 x64 and Windows 8.1 x64). 

For more details please see https://bugzilla.mozilla.org/show_bug.cgi?id=1279925#c53 and https://bugzilla.mozilla.org/show_bug.cgi?id=1279925#c44

Steps to reproduce:
1. Open page : https://bugzilla.mozilla.org/attachment.cgi?id=8762563

Actual results:

The characters ː and ˈ end up overlapping with the next character that is entered. It looks almost as though they are treated like Right-to-Left characters. 

Expected results:

The characters should show up in the sequence in which they are entered. These are not combining characters and should not overlap.

Comment 1

[Vincent Liu[:vliu]]

ni? Jonathan to have a look for this bug.

Comment 2

[Chris Peterson [:cpeterson]]

Timea, when you said in bug 1279925 comment 44 that this bug affects Windows 10 x64, are you testing 64-bit Firefox and/or 32-bit Firefox builds? I wonder whether this bug affects any Firefox running on 64-bit Windows OS or just 64-bit Firefox builds.

Comment 3

[Timea Zsoldos [:zstimi/tzsoldos], Desktop QA]

Yes, I tested both Firefox builds, 64-bit and 32-bit too, and I confirm that the issue is reproduce on 32-bit Firefox builds too.

Comment 4

[Jonathan Kew [:jfkthame]]

This is dependent on the exact version of Tahoma installed on the system, not on the specific Firefox build.

A workaround is included in harfbuzz 1.4.6, which was just released a few hours ago; so we need to update the library and that should resolve the problem.

Comment 5

[Jonathan Kew [:jfkthame]]

Attachment: Update harfbuzz to release 1.4.6

Pretty minor HB update; includes the GDEF blacklist entries for additional versions of Tahoma to fix the problem reported here.

Comment 7

[Chris Peterson [:cpeterson]]

(In reply to Timea Zsoldos from comment #3)
> Yes, I tested both Firefox builds, 64-bit and 32-bit too, and I confirm that
> the issue is reproduce on 32-bit Firefox builds too.

Thanks. That means this bug doesn't need to block our rollout of 64-bit Firefox builds to more users.

btw, harfbuzz 1.4.6 also includes the upstream fixes for clang's -Wcomma compiler warnings from bug 1356843 comment 61.

Comment 8

[Jonathan Kew [:jfkthame]]

review ping...?

Comment 9

[Jonathan Kew [:jfkthame]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/af1ace67caaa08555f37799bf0b837b0a7a497a9
Bug 1358502 - Update harfbuzz to release 1.4.6. r=jrmuizel

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/af1ace67caaa

Comment 11

[Ryan VanderMeulen [:RyanVM]]

Ready for Beta/ESR52 approval requests? :)

Comment 12

[Timea Zsoldos [:zstimi/tzsoldos], Desktop QA]

I can confirm this issue is fixed, I verified using Firefox 55.0a1 (both Firefox build 64-bit and 32-bit) on Win 8.1 x64 and Win 10 x64.

Comment 13

[Jonathan Kew [:jfkthame]]

Comment on attachment 8860888 [details] [diff] [review]
Update harfbuzz to release 1.4.6

Approval Request Comment
[Feature/Bug causing the regression]: bad fonts shipped with specific Windows versions
[User impact if declined]: buggy rendering of certain characters in Tahoma font
[Is this code covered by automated tests?]: it's exercised in the course of font/text tests, but the specific fixes here are not explicitly tested (too dependent on specific installed font versions)
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: just minor changes in new upstream lib version
[String changes made/needed]: none

Comment 14

[Jonathan Kew [:jfkthame]]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #11)
> Ready for Beta/ESR52 approval requests? :)

I marked it for approval-beta? consideration; I doubt this really meets the criteria for ESR uplift, does it?

Comment 15

[Gerry Chang [:gchang]]

Comment on attachment 8860888 [details] [diff] [review]
Update harfbuzz to release 1.4.6

Fix certain characters in Tahoma font rendering issue and was verified. Beta54+. Should be in 54 beta 7.

Comment 16

[Ryan VanderMeulen [:RyanVM]]

ESR52 is on version 1.4.1 still, so yeah, I guess I agree with you on this if we haven't done other updates either.

Comment 17

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/9c3477794d33

Comment 18

[Andrei Vaida [:avaida]]

(In reply to Jonathan Kew (:jfkthame) from comment #13)
> [Is this code covered by automated tests?]: it's exercised in the course of
> font/text tests, but the specific fixes here are not explicitly tested (too
> dependent on specific installed font versions)
> [Has the fix been verified in Nightly?]: yes
> [Needs manual test from QE? If yes, steps to reproduce]: no

Setting qe-verify- based on Jonathan's assessment on manual testing needs and the fact that the area affected by this fix has automated coverage.