Re: [9] Review request for 8169133: This time, on Windows: java/awt/Robot/SpuriousMouseEvents/SpuriousMouseEvents.java

2016-11-25 Thread Semyon Sadetsky
On 11/25/2016 4:13 PM, Sergey Bylokhov wrote: On 24.11.16 22:21, Semyon Sadetsky wrote: All robots may be used to address any area of the desktop regardless of the screen constructor argument. The coordinate system is the only aspect the screen argument affects. This statement means that

Re: [9] Review request for 8169589: [macosx] Activating a JDialog puts to back another dialog

2016-11-25 Thread Sergey Bylokhov
Hi, Dmitry. Can you please adds some comments to the code and describe what is going on. On 25.11.16 16:08, dmitry markov wrote: Hello, Could you review a fix for jdk9, please? bug: https://bugs.openjdk.java.net/browse/JDK-8169589 webrev:

Re: [9] Review request for 8169133: This time, on Windows: java/awt/Robot/SpuriousMouseEvents/SpuriousMouseEvents.java

2016-11-25 Thread Sergey Bylokhov
On 24.11.16 22:21, Semyon Sadetsky wrote: All robots may be used to address any area of the desktop regardless of the screen constructor argument. The coordinate system is the only aspect the screen argument affects. This statement means that this constructor will have any effects when used

[9] Review request for 8169589: [macosx] Activating a JDialog puts to back another dialog

2016-11-25 Thread dmitry markov
Hello, Could you review a fix for jdk9, please? bug: https://bugs.openjdk.java.net/browse/JDK-8169589 webrev: http://cr.openjdk.java.net/~dmarkov/8169589/webrev.00/ Problem description: Current implementation of CPlatformWindow.orderAboveSiblings() just recursively pops up the windows