<AWT Dev> [11] Review Request: JDK-8196435 Regression automated Test 'java/awt/Mouse/GetMousePositionTest/GetMousePositionWithOverlay.java' fails

Prem Balakrishnan prem.balakrishnan at oracle.com
Fri Mar 2 04:16:38 UTC 2018


Hi Sergey,

The existing waitForIdle() call at line 55 is called after calling setVisible() for both the frames (which I have removed in proposed fix). I have added waitForIdle() call after calling setVisible() for each frame(at line 103 and 110 in proposed fix), which ensures frontFrame is rendered on top of backFrame.

Regards,
Prem

-----Original Message-----
From: Sergey Bylokhov 
Sent: Friday, March 02, 2018 3:19 AM
To: Prem Balakrishnan <prem.balakrishnan at oracle.com>; awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> [11] Review Request: JDK-8196435 Regression automated Test 'java/awt/Mouse/GetMousePositionTest/GetMousePositionWithOverlay.java' fails

Hi, Prem.
On 26/02/2018 21:20, Prem Balakrishnan wrote:
> At times backFrame is rendered on top of frontFrame, due to which mouse position on backFrame is not NULL and test fails, solved this by adding waitForIdle() calls appropriately. Furthermore, mousemove from frontFrame to backFrame, caused the cursor to change to resize cursor and test fails, solved this by setting setResizable() to false.

But the old version of the test also call Util.waitForIdle(null) after frames were shown at line 55.


-- 
Best regards, Sergey.


More information about the awt-dev mailing list