Re: [9] Review Request for 8130390: Applet fails to launch on virtual desktop

2015-07-08 Thread Sergey Bylokhov
The fix looks fine. On 08.07.15 14:48, Semyon Sadetsky wrote: Sergey, thanks, again! http://cr.openjdk.java.net/~ssadetsky/8130390/webrev.02/ --Semyon On 7/8/2015 2:35 PM, Sergey Bylokhov wrote: On 08.07.15 12:37, Semyon Sadetsky wrote: Thanks, Sergey! The update fix: http://cr.openjdk.java

Re: [9] Review Request for 8130390: Applet fails to launch on virtual desktop

2015-07-08 Thread Semyon Sadetsky
Sergey, thanks, again! http://cr.openjdk.java.net/~ssadetsky/8130390/webrev.02/ --Semyon On 7/8/2015 2:35 PM, Sergey Bylokhov wrote: On 08.07.15 12:37, Semyon Sadetsky wrote: Thanks, Sergey! The update fix: http://cr.openjdk.java.net/~ssadetsky/8130390/webrev.01/ You forgot to update the java

Re: [9] Review Request for 8130390: Applet fails to launch on virtual desktop

2015-07-08 Thread Sergey Bylokhov
On 08.07.15 12:37, Semyon Sadetsky wrote: Thanks, Sergey! The update fix: http://cr.openjdk.java.net/~ssadetsky/8130390/webrev.01/ You forgot to update the javadoc for class itself: * Inner class for flipping buffers on a component. That component must * be a Canvas or Window. --

Re: [9] Review Request for 8130390: Applet fails to launch on virtual desktop

2015-07-08 Thread Semyon Sadetsky
Thanks, Sergey! The update fix: http://cr.openjdk.java.net/~ssadetsky/8130390/webrev.01/ --Semyon On 7/7/2015 9:46 PM, Sergey Bylokhov wrote: Hi, Semyon. It seems that the javadoc of this method/class should be updated via ccc? On 07.07.15 13:38, Semyon Sadetsky wrote: Hello, Please review f

Re: [9] Review Request for 8130390: Applet fails to launch on virtual desktop

2015-07-07 Thread Sergey Bylokhov
Hi, Semyon. It seems that the javadoc of this method/class should be updated via ccc? On 07.07.15 13:38, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8130390 webrev: http://cr.openjdk.java.net/~ssadetsky/8130390/webrev.00/ The FlipBuf

Re: [9] Review Request for 8130390: Applet fails to launch on virtual desktop

2015-07-07 Thread Alexander Zvegintsev
looks fine to me. Thanks, Alexander. On 07/07/2015 01:38 PM, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8130390 webrev: http://cr.openjdk.java.net/~ssadetsky/8130390/webrev.00/ The FlipBufferStrategy did not take an Applet possibi

[9] Review Request for 8130390: Applet fails to launch on virtual desktop

2015-07-07 Thread Semyon Sadetsky
Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8130390 webrev: http://cr.openjdk.java.net/~ssadetsky/8130390/webrev.00/ The FlipBufferStrategy did not take an Applet possibility into account. --Semyon