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.balakrish...@oracle.com>; awt-dev@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.

Reply via email to