Re: AWT Dev Swing Dev [9] Review Request:JDK-8130481 [TEST_BUG] javax/swing/JPopupMenu/6583251/bug6583251.java failed with UnsupportedOperation exception

2015-08-27 Thread Alexander Zvegintsev
BTW, there is no need in try/catch block. Thanks, Alexander. On 08/27/2015 03:08 PM, Alexander Scherbatiy wrote: On 8/27/2015 1:12 PM, pooja chopra wrote: Hi Alexandr, Please review updated webrev link :- http://cr.openjdk.java.net/~pchopra/8130481/webrev.06/ There are some problems

AWT Dev [9] Review Request: 8134603 Incorrect destination is used in CGLLayer surface

2015-08-27 Thread Sergey Bylokhov
Hello. Please review the fix for jdk9. The CGLLayer.java has a typo it uses the peer as a destination, but the target should be used instead. This breaks the bufferPerWIndow functionality, because this destination is used in BufferStrategyPaintManager. Bug:

Re: AWT Dev Swing Dev [9] Review Request:JDK-8130481 [TEST_BUG] javax/swing/JPopupMenu/6583251/bug6583251.java failed with UnsupportedOperation exception

2015-08-27 Thread pooja chopra
Hi Alexandr, Please review updated webrev link :- http://cr.openjdk.java.net/~pchopra/8130481/webrev.06/ Regards, Pooja On 8/5/2015 2:46 PM, Alexander Scherbatiy wrote: On 8/5/2015 10:31 AM, pooja chopra wrote: Hi Alexandr, Please review update webrev link :- The webrev is :

Re: AWT Dev Swing Dev [9] Review Request:JDK-8130481 [TEST_BUG] javax/swing/JPopupMenu/6583251/bug6583251.java failed with UnsupportedOperation exception

2015-08-27 Thread Alexander Scherbatiy
On 8/27/2015 1:12 PM, pooja chopra wrote: Hi Alexandr, Please review updated webrev link :- http://cr.openjdk.java.net/~pchopra/8130481/webrev.06/ There are some problems with the code indentation like : -- if (SystemTray.isSupported()) {