[OpenJDK 2D-Dev] [11] Review Request: 8198406 Test TestAATMorxFont is unstable

[Semyon Sadetsky]

On 03/08/2018 12:47 PM, Sergey Bylokhov wrote:

> On 08/03/2018 12:38, Semyon Sadetsky wrote:
>>> This check was added for the case when the test will be run in 
>>> standalone w/o jtreg
>> That's doesn't make any sense. The w/o jtreg it is run manually and 
>> it is always known on what is the platform. There may be a need to 
>> run the test on another platform but this redundant check makes it 
>> not possible.
>
> It was intentional decision, if this test will be run on other 
> platform it will pass because it is targeted to macOS.
Then for what you have added platform filter in the jtreg header?
>
>>>
>>>>
>>>> --Semyon
>>>>
>>>>
>>>> On 03/07/2018 07:32 PM, Sergey Bylokhov wrote:
>>>>> Hello.
>>>>> Please review the fix for jdk11.
>>>>>
>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8198406
>>>>> Webrev can be found at: 
>>>>> http://cr.openjdk.java.net/~serb/8198406/webrev.00
>>>>>
>>>>> Initially, this test was marked as unstable, because sometimes it 
>>>>> passed(when invokeLater did not start till the end of the main 
>>>>> method) or failed when the code in invokeLater thrown an 
>>>>> HeadlessException.
>>>>>
>>>>> In the fix the test is reworked to use BufferedImage instead of 
>>>>> Frame, so it will work in headful and headless environment.
>>>>>
>>>>> Some additional info:
>>>>> http://mail.openjdk.java.net/pipermail/2d-dev/2018-February/008947.html 
>>>>>
>>>>>
>>>>
>>>
>>>
>>
>
>