The fix looks good to me.
Thanks,
Alexandr.
On 14/03/16 16:25, Avik Niyogi wrote:
Hi All,
Please review code changes made as per inputs provided.
http://cr.openjdk.java.net/~aniyogi/8151282/webrev.01/
<http://cr.openjdk.java.net/%7Eaniyogi/8151282/webrev.01/>
With Regards,
Avik Niyogi
On 14-Mar-2016, at 10:53 am, Avik Niyogi <avik.niy...@oracle.com
<mailto:avik.niy...@oracle.com>> wrote:
Hi Sergey,
Seems like it is a delay issue. Will submit test case with a
waitForIdle() fix.
With Regards,
Avik Niyogi
On 10-Mar-2016, at 7:22 pm, Sergey Bylokhov
<sergey.bylok...@oracle.com <mailto:sergey.bylok...@oracle.com>> wrote:
Hi, Avik.
A few questions.
- Why webrev contains the new file?
- You marked the test as a mac specific but it is iterates over a
bunch of l&fs. It seems it should not be OS specific, but should
check some specific l&fs (which support such icons): Metal, Nimbus,
Aqua, Windows(???). So gtk/motif should be skipped.
On 08.03.16 8:10, Avik Niyogi wrote:
Hi All,
Kindly review the bug fix for JDK 9.
*Bug:*
https://bugs.openjdk.java.net/browse/JDK-8151282
*Webrev:*
_http://cr.openjdk.java.net/~aniyogi/8151282/webrev.00/_
_
_
*Issue:*
The test case failed for GTK LAF.
*Cause:*
The test case was meant to be Mac specific and was missing a jtreg
parameter
*Fix:*
Minor change to test case with @requires tag added to set the fix.
With Regards,
Avik Niyogi
--
Best regards, Sergey.
