<Swing Dev> [14] RFR JDK-8229935:[TEST_BUG]: bug8132119.java inconsistently positions text

[Sergey Bylokhov]

Looks fine.

On 11/4/19 10:08 pm, Prasanta Sadhukhan wrote:
> I have updated the webrev removing the problemlisting
> 
> http://cr.openjdk.java.net/~psadhukhan/8229935/webrev.1/
> 
> Regards
> Prasanta
> On 05-Nov-19 2:42 AM, Sergey Bylokhov wrote:
>> Hi, Prasanta.
>>
>> It looks like this test was problem listed[1], but unfortunately it was pushed to jdk/jdk instead of jdk/client.
>> So you need to wait the merge jdk->client to update the webrev.
>>
>> [1] https://hg.openjdk.java.net/jdk/jdk/rev/75eedcd30a2a
>>
>> On 11/4/19 11:44 am, Sergey Bylokhov wrote:
>>> Looks fine.
>>>
>>> On 11/4/19 8:29 am, Prasanta Sadhukhan wrote:
>>>> Hi All,
>>>>
>>>> Please review a test fix for an  test issue seen primarily in linux.
>>>>
>>>> The test draws "AOB" with and without underline underneath and then looks to try to calculate the midpoint of "O" and then expects to see the transition
>>>> from top to bottom of bg->fg->bg->fg->bg as you get when you cross the path of the O twice. When there is underline, it expects 6 transition from top of O till it passes the underline in form of bg->fg->bg->fg->bg->fg->bg
>>>>
>>>> as it apparently assumes that there is at least a pixel gap between lower curvature of O to underline. This may not be true for all fonts, so sometimes underline are stuck with the lower part of O and it fails.
>>>>
>>>> Proposed fix is to calculate the transition from 10 pixel left from midpoint of "O" so that there is at least some pixels gap between curvature of O and underline.
>>>>
>>>> Also, the text is drawn in aliased mode to avoid the antialias zigzag to be mistakenly construed as "transition".
>>>>
>>>> Proposed fix pass in all platforms and the mach5 job link is present in JBS.
>>>>
>>>> Bug:https://bugs.openjdk.java.net/browse/JDK-8229935
>>>>
>>>> webrev: http://cr.openjdk.java.net/~psadhukhan/8229935/webrev.0/
>>>>
>>>> Regards
>>>> Prasanta
>>>
>>>
>>
>>


-- 
Best regards, Sergey.