Re: Review Request of 8146321: [macosx] JInternalFrame frame icon in wrong position on Mac L&F if icon is not ImageIcon

2016-02-01 Thread Avik Niyogi
Hi All, Please review the code changes made as per inputs provided: cr.openjdk.java.net/~aniyogi/8146321/webrev.02 With Regards, Avik Niyogi > On 20-Jan-2016, at 10:35 pm, Sergey Bylokhov > wrote: > > On 20/01/16 18:43, Avik Niyogi wrote:

Re: Review request for JDK-8131751: Test javax/swing/plaf/gtk/crash/RenderBadPictureCrash.java fails UnsupportedOperationException

2016-02-01 Thread Alexandr Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 2/1/2016 4:32 AM, Vikrant Agarwal wrote: Thanks Alexander, I have updated the webrev for the same. Updated Webrev: http://cr.openjdk.java.net/~ntv/vikrant/8131751/webrev.01/

Re: Review request for JDK-8131751: Test javax/swing/plaf/gtk/crash/RenderBadPictureCrash.java fails UnsupportedOperationException

2016-02-01 Thread Vikrant Agarwal
Thanks Alexander, I have updated the webrev for the same. Updated Webrev: http://cr.openjdk.java.net/~ntv/vikrant/8131751/webrev.01/ Bug: https://bugs.openjdk.java.net/browse/JDK-8131751 Best Regards, Vikrant From: Alexander Scherbatiy Sent: Thursday, January 28, 2016 11:13 PM To: Vikr

Re: Review Request for 8062846 : Transparent JDialog will lose transparency upon iconify/deiconify sequence.

2016-02-01 Thread Alexandr Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 1/28/2016 9:45 PM, Prem Balakrishnan wrote: Hi*,* Please review fix for JDK9, *Bug: *https://bugs.openjdk.java.net/browse/JDK-8062946** *Webrev: *http://cr.openjdk.java.net/~arapte/prem/8062946/webrev.00/