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








Reply via email to