Re: [9] Fix for JDK-6567433 : JComponent.updateUI() may create StackOverflowError

2016-07-12 Thread Ajit Ghaisas
Hi Andrej, Can you please review this? Regards, Ajit -Original Message- From: Ajit Ghaisas Sent: Friday, July 08, 2016 10:32 AM To: Andrej Golovnin Cc: Rajeev Chamyal; Alexander Scherbatiy; swing-dev@openjdk.java.net Subject: RE: [9] Fix for JDK-6567433 : JComponent.updateUI() may

Re: [9] Fix for JDK-6567433 : JComponent.updateUI() may create StackOverflowError

2016-07-12 Thread Alexandr Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 7/12/2016 3:58 PM, Andrej Golovnin wrote: Hi Ajit, it looks good for me. Thanks! And you need a reviewer from the Swing team as I don’t have the reviewer role. Best regards, Andrej Golovnin -Original Message- From: Ajit Ghaisas Sent:

Re: [9] Fix for JDK-6567433 : JComponent.updateUI() may create StackOverflowError

2016-07-12 Thread Andrej Golovnin
Hi Ajit, it looks good for me. Thanks! And you need a reviewer from the Swing team as I don’t have the reviewer role. Best regards, Andrej Golovnin > > -Original Message- > From: Ajit Ghaisas > Sent: Friday, July 08, 2016 10:32 AM > To: Andrej Golovnin > Cc: Rajeev Chamyal; Alexander

Re: 8160438: [PIT][macosx] [TEST_BUG] javax/swing/plaf/nimbus/8057791/bug8057791.java fails

2016-07-12 Thread Alexandr Scherbatiy
On 7/12/2016 8:24 AM, Avik Niyogi wrote: Hi All, A gentle reminder, please review my code changes. With Regards, Avik Niyogi On 08-Jul-2016, at 4:24 pm, Yuri Nesterenko wrote: It pass for me if I provide some time to process native events after the user activity