[11u] RFR: 8220231: Cache HarfBuzz face object for same font's text layout calls

[Lindenmaier, Goetz]

Hi Paul,

thanks for reviewing!

Best regards,
  Goetz.

> -----Original Message-----
> From: Hohensee, Paul <hohensee at amazon.com>
> Sent: Montag, 18. November 2019 18:51
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; jdk-updates-
> dev at openjdk.java.net
> Subject: Re: [11u] RFR: 8220231: Cache HarfBuzz face object for same font's
> text layout calls
> 
> Lgtm.
> 
> Paul
> 
> On 11/18/19, 4:52 AM, "jdk-updates-dev on behalf of Lindenmaier, Goetz"
> <jdk-updates-dev-bounces at openjdk.java.net on behalf of
> goetz.lindenmaier at sap.com> wrote:
> 
>     Hi,
> 
>     i would like to downport this change for parity with 11.0.6-oracle.
>     I had to resolve two files, both trivial:
> 
>     src/java.desktop/share/classes/sun/font/FreetypeFontScaler.java
>     One patch did not apply due to a difference in a comment in the context.
> 
>     src/java.desktop/share/native/libfontmanager/hb-jdk-font.cc
>     A huge chunk could not get applied. This is because tiny
>     "8223886: Add in font table referene" was applied after
>     this change in jdk/jdk, but was already downported to 11u-dev,
>     so it comes there first.
>     The version history of this file in 11 and 14 is exactly the
>     same, I resolved the conflict so that the files look alike.
> 
>     webrev: http://cr.openjdk.java.net/~goetz/wr19/8220231-cache_harfbuzz-
> jdk11/01/
>     bug: https://bugs.openjdk.java.net/browse/JDK-8220231
>     orig change: http://hg.openjdk.java.net/jdk/jdk/rev/98473958d49a
>     The conflicting change is closed, unfortunately:
>     8223886: Add in font table referene
>     http://hg.openjdk.java.net/jdk-updates/jdk11u-dev/rev/174afa5802bd
> 
>     Best regards,
>       Goetz
>