Re: [9] Review request for JDK-8150176 [hidpi] wrong resolution variant of multi-res. image is used for TrayIcon

2016-06-13 Thread Rajeev Chamyal
Hello Alexandr, Thanks for the review. I have updated the webrev as per review comments. http://cr.openjdk.java.net/~rchamyal/8150176/webrev.01/ I tried drawing the image directly to paint graphics without buffered image and it was getting cropped. Regards, Rajeev Chamyal From:

Re: [9] RFR JDK-8159068:The rendering of JTable is broken

2016-06-13 Thread prasanta sadhukhan
On 6/13/2016 12:51 PM, prasanta sadhukhan wrote: Hi All, Please review a fix for jdk9 where it was seen that if we try to select some rows in a JTable, the text painted in the rows goes missing. Bug: https://bugs.openjdk.java.net/browse/JDK-8159068 webrev:

Re: Fix for JDK-8065861 : Pressing Esc does not set 'canceled' property of ProgressMonitor

2016-06-13 Thread Rajeev Chamyal
Looks good to me. Regards, Rajeev Chamyal -Original Message- From: Alexandr Scherbatiy Sent: 10 June 2016 13:06 To: Ajit Ghaisas; Sergey Bylokhov; Rajeev Chamyal; swing-dev@openjdk.java.net Subject: Re: Fix for JDK-8065861 : Pressing Esc does not set 'canceled' property of

[9] RFR JDK-8159068:The rendering of JTable is broken

2016-06-13 Thread prasanta sadhukhan
Hi All, Please review a fix for jdk9 where it was seen that if we try to select some rows in a JTable, the text painted in the rows goes missing. Bug: https://bugs.openjdk.java.net/browse/JDK-8159068 webrev: http://cr.openjdk.java.net/~psadhukhan/8159068/webrev.00/ The issue was rMax value

Re: [9] Review request for JDK-8159135 [PIT] javax/swing/JMenuItem/8152981/MenuItemIconTest.java always fail

2016-06-13 Thread Rajeev Chamyal
Hello Alexandr, Please review the updated webrev. http://cr.openjdk.java.net/~rchamyal/8159135/webrev.01/ Regards, Rajeev Chamyal From: Alexandr Scherbatiy Sent: 10 June 2016 19:52 To: Rajeev Chamyal; Sergey Bylokhov; swing-dev@openjdk.java.net Subject: Re: [9] Review request for