On 03/08/2018 12:03 PM, Sergey Bylokhov wrote:
On 08/03/2018 08:30, Semyon Sadetsky wrote:
Please remove redundant platform detection code since this is done in
the jtreg header now.
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.
--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
