Hi Krishna,
Does the line
EventQueue.invokeAndWait(() -> { /* Empty */ });
is really necessary for test to pass?
--Semyon
On 02/27/2018 09:56 AM, Krishna Addepalli wrote:
Hi Sergey/Phil,
Thanks for pointing that out. I have created a new webrev:
http://cr.openjdk.java.net/~kaddepalli/8197808/webrev01/
with the appropriate changes for ProblemList.txt as well.
I have tested on Windows 7, Ubuntu 16.04, Ubuntu 17.10 and MacOS 10.13, and the
test passed with/without my changes.
As noted earlier, on Windows 10, the test passes after the fix.
Thanks,
Krishna
-----Original Message-----
From: Philip Race
Sent: Tuesday, February 27, 2018 6:01 AM
To: Sergey Bylokhov <sergey.bylok...@oracle.com>
Cc: Krishna Addepalli <krishna.addepa...@oracle.com>; awt-dev@openjdk.java.net
Subject: Re: <AWT Dev> [11][JDK-8197808] Test
java/awt/Dialog/MakeWindowAlwaysOnTop/MakeWindowAlwaysOnTop.java fails on Windows
Yes .. even if bugs were added to the problem list in bulk, the expectation is
that they'll be removed by the bug fix that fixes the failure.
And if it still fails on some platforms then either (1) you aren't done with
your fix, or (2) you need to find a different bug ID under which to now exclude
the test for the platform(s) that still fail for what would normally be some
completely un-related reason.
Other than that the fix is fine.
-phil.
On 2/26/18, 2:16 PM, Sergey Bylokhov wrote:
Hi, Krishna.
This test was added to the problemlist, please check that it works on
other platforms and remove it from the problemlist.
On 25/02/2018 17:53, Krishna Addepalli wrote:
Hi All,
Please review a fix for JDK-8197808:
https://bugs.openjdk.java.net/browse/JDK-8197808
Webrev: http://cr.openjdk.java.net/~kaddepalli/8197808/webrev00/
The problem was that by the time Robot tries to read the color on
screen, the compositing of the window is not completed, and hence,
instead of reading Blue color, it is reading a combination of Blue
and Red colors. Increasing the delay by 100 ms made sure that the
composition is complete and the color is being read correctly. Also,
this happens only on Windows 10.
Apart from this, the test was not cleaning up the windows when it
throws exception, so fixed that, and also added all explicit imports.
Thanks,
Krishna
