Hello All, Please review the following webrev. I have updated the fix as per review comments.
http://cr.openjdk.java.net/~psadhukhan/rajeev/8032568/webrev.01/ Regards, Rajeev Chamyal -----Original Message----- From: Phil Race Sent: Thursday, September 03, 2015 11:26 PM To: Rajeev Chamyal Cc: Alexander Scherbatiy; Alexander Zvegintsev; swing-dev@openjdk.java.net Subject: Re: <Swing Dev> Request for review fo bug JDK-8032568 Test javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual hosts Could you instead use https://docs.oracle.com/javase/8/docs/api/javax/swing/AbstractButton.html#getMnemonic-- ? Of course you need to get the button itself first rather than the string. -phil. On 09/03/2015 08:39 AM, Rajeev Chamyal wrote: > Hello All, > > I need one more review for this fix. Can someone please review it. > > Webrev : http://cr.openjdk.java.net/~psadhukhan/rajeev/8032568/webrev.00/ > > Regards, > Rajeev Chamyal > > -----Original Message----- > From: Sergey Bylokhov > Sent: Thursday, September 03, 2015 9:05 PM > To: Rajeev Chamyal; Alexander Scherbatiy; Alexander Zvegintsev; > swing-dev@openjdk.java.net > Subject: Re: Request for review fo bug JDK-8032568 Test > javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual > hosts > > The fix looks good. > > On 03.09.15 9:15, Rajeev Chamyal wrote: >> Hi, >> >> Please review the following fix for jdk9: >> >> Bug:https://bugs.openjdk.java.net/browse/JDK-8032568 >> >> Webrev : http://cr.openjdk.java.net/~psadhukhan/rajeev/8032568/webrev.00/ >> >> Translation team has changed the JInternalFrame close mnemonic character >> from C to different character for different languages. >> >> For example in German close mnemonic is now S and for French its now F. To >> fix the issue we are now finding the correct mnemonic character and passing >> it to robo. >> >> Regards, >> >> Rajeev Chamyal >> >