[OpenJDK 2D-Dev] [7u6] request for review: 7120895: FontConfiguration should not use thread contextClassLoader

[Igor Nekrestyanov]

Looks fine to me.

-igor

On 5/31/12 1:39 AM, Andrew Brygin wrote:
> Hello,
>
>  could you please review a backport of the fix for 7120895 to 7u6?
>
>  The problem there seems to be caused by an attempt to load
>  sun.awt.HKSCS class using thread context classloader, that
>  leads to loading lazy jars in case of webstart application,
>  and has noticeable performance impact.
>
>  The problem can be resolved by using system class loader instead
>  of thread context class loader (FontConfiguration.
>  getFontCharsetEncoder, lines 949 - 950).
>  Note that we use this branch of code just to resolve "sun.awt.*"
>  charset names, so we can limit the scope to search by system
>  classes only.
>
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7120895
> jdk8 changeset: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/d988ed9d40bc
> Webrev: http://cr.openjdk.java.net/~bae/7120895/7u6/webrev.00/
>
> Thanks,
> Andrew