[OpenJDK 2D-Dev] RFR: 8183978 : Remove ICU layout code from OpenJDK

[Phil Race]

I am not sure that the test is any less valid .. it has recorded 
specific results of layout with ICU
and runs harfbuzz comparing it to the archived results.

But even as a manual test it will be a no-op without the right font 
environment.

But I'll leave this up to Steven to decide as he provided us with these 
tests.

One other issue with this test is that it uses JAXP classes which are 
slated to be removed ..
Perhaps jtreg will still provide a way to access these classes on the 
classpath without
test modifications  ???

-phil.

On 10/12/2017 02:22 PM, Sergey Bylokhov wrote:
> Looks fine.
> I guess we should update TestLayoutVsICU.java test which was added as 
> part of harfbuzz integration. I assumed that it should compares 
> behavior against icu, but it seems it is never executed.
>
> On 12/10/2017 10:45, Steven R. Loomis wrote:
>> approve
>>
>> On Wed, Oct 11, 2017 at 9:59 AM, Phil Race <philip.race at oracle.com 
>> <mailto:philip.race at oracle.com>> wrote:
>>
>>     Bug: https://bugs.openjdk.java.net/browse/JDK-8183978
>>     <https://bugs.openjdk.java.net/browse/JDK-8183978>
>>     Webrev: http://cr.openjdk.java.net/~prr/8183978/
>>     <http://cr.openjdk.java.net/~prr/8183978/>
>>
>>     This fix removes the obsolete ICU opentype layout code from JDK 10.
>>
>>     I've built this on all platforms (including open+closed builds).
>>
>>     -phil.
>>
>>
>
>