<AWT Dev> [11][JDK-8197808] Test java/awt/Dialog/MakeWindowAlwaysOnTop/MakeWindowAlwaysOnTop.java fails on Windows

Philip Race philip.race at oracle.com
Tue Feb 27 00:32:36 UTC 2018



On 2/26/18, 9:22 AM, Manajit Halder wrote:
> Hi Krishna,
>
> Fix looks good.
>
> I suggest the following changes before committing.
>  1) Remove the comma in between the two bug ids.
yes .. not sure if jtreg cares though.
>  2) Is the message “Test PASSED” required?

I find these statements useful .. silence doesn't tell you that it is 
done or succeeded.

-phil.
>
> Regards,
> Manajit
>
>
>> On 26-Feb-2018, at 10:23 PM, Semyon Sadetsky 
>> <semyon.sadetsky at oracle.com <mailto:semyon.sadetsky at oracle.com>> wrote:
>>
>> +1
>>
>> --Semyon
>>
>>
>> On 02/25/2018 05:53 PM, 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/ 
>>> <http://cr.openjdk.java.net/%7Ekaddepalli/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
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20180226/3cfed29b/attachment.html>


More information about the awt-dev mailing list