[OpenJDK 2D-Dev] [9]: RFR JDK-8023213 [macosx] closed/java/awt/FontClass/NaNTransform.java fails on MacOS X 10.9

[prasanta sadhukhan]

Hi Phil,

I have modified as per your review to populate GlyphInfo with "empty" glyph
and also moved your existing test to "open"
I also added a Infinite Transform test along with your NaN transform 
just incase (in fact Sergey informally asked me to check)

http://cr.openjdk.java.net/~psadhukhan/8023213/webrev.01/

Regards
Prasanta
On 10/30/2015 1:03 AM, Phil Race wrote:
> Should this new check go before this :
>
> CGGI_ClearCanvas(canvas, info);
>
> since it is using info which is where you get NaN from.
>
>
> And should we then populate the returned canvas and info to
> ensure that we return an "empty" glyph rather than garbage values ?
> It is not clear to me that this is occurring.
>
> Why does the bug report not contain the evaluation below ?
> Also why is there a new test ? I would expect SQE would
> want to run the existing test to verify this fix.
> Should we not just open the existing test ?
>
>
> -phil.
>
> On 10/13/2015 04:49 AM, prasanta sadhukhan wrote:
>> Gentle reminder for review
>>
>> Regards
>> Prasanta
>> On 10/6/2015 3:25 PM, prasanta sadhukhan wrote:
>>> Hello All,
>>>
>>> Please review a fix for jdk9
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8023213
>>> webrev: http://cr.openjdk.java.net/~psadhukhan/8023213/webrev.00/
>>>
>>> drawString takes long time in mac native call 
>>> CGContextShowGlyphsAtPoint() if NaN transform is used which 
>>> translated to x & y being NaN.
>>> Fix is to prevent calling mac api for such invalid usage.
>>>
>>> Regards
>>> Prasanta
>>
>