Re: [9] Review request for JDK-8160650: Couple awt and swing tests have wrong require jtreg arguments

2016-06-30 Thread Alexandr Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 6/30/2016 4:01 PM, mikhail cherkasov wrote: Hi all, jbs: https://bugs.openjdk.java.net/browse/JDK-8160650 webrev:http://cr.openjdk.java.net/~mcherkas/8160650/9/webrev/ Couple awt and swing tests have wrong require jtreg arguments, the following t

Re: [9] Review request for JDK-8160650: Couple awt and swing tests have wrong require jtreg arguments

2016-06-30 Thread Sergey Bylokhov
Looks fine. On 30.06.16 16:01, mikhail cherkasov wrote: Hi all, jbs: https://bugs.openjdk.java.net/browse/JDK-8160650 webrev:http://cr.openjdk.java.net/~mcherkas/8160650/9/webrev/ Couple awt and swing tests have wrong require jtreg arguments, the following tests should use "os.family" instead

[9] Review request for JDK-8160650: Couple awt and swing tests have wrong require jtreg arguments

2016-06-30 Thread mikhail cherkasov
Hi all, jbs: https://bugs.openjdk.java.net/browse/JDK-8160650 webrev:http://cr.openjdk.java.net/~mcherkas/8160650/9/webrev/ Couple awt and swing tests have wrong require jtreg arguments, the following tests should use "os.family" instead of "os.name": javax/swing/LookAndFeel/8145547/DemandGTK.j