<AWT Dev> Review request for 8041928: MouseEvent.getModifiersEx gives wrong result

[Sergey Bylokhov]

Hi, Ambarish.
Fix looks fine. Can you please confirm that combination like AltGr+C → © 
(copyright sign) works in our text components?

On 09/12/15 18:30, Ambarish Rapte wrote:
> Hi,
>
>                  Please review the fix for JDK9
>
>                  Bug: https://bugs.openjdk.java.net/browse/JDK-8041928
>
> Webrev: http://cr.openjdk.java.net/~arapte/8041928/webrev.00/
>
> Issue:
>
> The 8041928 issue mentions below problems with the modifiers,
>
> 1.When clicking right button it gives Button3 and the Meta key modifiers.
>
> 2.Alt-Gr modifier is not set.
>
> Cause:
>
> 1.When clicking right button it gives Button3 and the Meta key modifiers.
>
> This is actually not an issue, the right mouse button (BUTTON3_MASK)
> click and META key  (META_MASK) share same bit in modifiers.
>
> Please check documentation here,
>
> https://docs.oracle.com/javase/8/docs/api/java/awt/event/MouseEvent.html
>
> The important statement from this doc:
>
> (Note: Due to overlap in the values of ALT_MASK/BUTTON2_MASK and
> META_MASK/BUTTON3_MASK, this is not always true for mouse events
> involving modifier keys)
>
> 2.Alt-Gr modifier is not set.
>
> This is an issue, and the patch fixes this issue for Windows.
>
> Also added related manual test.
>
> This bug does not happen on Linux,
>
> But occurs with MACOSX,  Shall raise a new MACOSX specific bug to
> continue work.
>
> Fix:
>
> Added check for key code VK_RMENU for windows, while creating the java
> Modifiers of event.
>
> The related test is manual because,
>
> 1.Robot cannot generate the Alt-Gr key event
>
> 2.There are different keyboard layouts, A keyboard may have Alt-Gr key
> or might have to use Ctrl-Alt combination.
>
> 3.For Linux – ubuntu, The alternate graphics key needs to be enabled
> explicitly from System Keyboard settings.
>
> Thanks,
>
> Ambarish
>


-- 
Best regards, Sergey.