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.

Reply via email to