Looks fine to me too.
On 19.05.15 17:09, Alexander Zvegintsev wrote:
Hello Alexandr,
the fix looks fine to me.
Thanks,
Alexander.
On 05/19/2015 05:06 PM, Alexander Scherbatiy wrote:
The test fails on Linux because Xsystem does not set state for
motion events when extra button is pressed.
I have filled the issue on it: JDK-8080676 Mouse drag does not work
for extra buttons on Linux
https://bugs.openjdk.java.net/browse/JDK-8080676
Thanks,
Alexandr.
On 5/19/2015 1:10 PM, Alexander Scherbatiy wrote:
Hello,
Could you review the fix:
bug: https://bugs.openjdk.java.net/browse/JDK-8080137
webrev: http://cr.openjdk.java.net/~alexsch/8080137/webrev.00
The LightweightDispatcher.isMouseGrab(MouseEvent) method does not
take into account the extended modifiers for additional buttons.
Thanks,
Alexandr.
--
Best regards, Sergey.
