Hi, The fix looks good to me. A small query though, line 281 - 290 is required at that position, looks like it was moved.
With Regards, Avik Niyogi > From: Manajit Halder > Sent: Monday, June 20, 2016 1:56 AM > To: Sergey Bylokhov; Semyon Sadetsky > Cc: awt-dev@openjdk.java.net <mailto:awt-dev@openjdk.java.net> > Subject: <AWT Dev> <AWT dev>[9] Review request for JDK-8156460 [macosx] Test > case javax/swing/JPopupMenu/6827786/bug6827786.java fails > > Hi All, > > Please review the regression fix for issue JDK-8156460 > <https://bugs.openjdk.java.net/browse/JDK-8156460> which fixes below > mentioned test cases. > http://cr.openjdk.java.net/~mhalder/8156460/webrev.00/ > <http://cr.openjdk.java.net/~mhalder/8156460/webrev.00/> > > This fix resolves the following 3 JCK failures and 7 test failures: > > JCK tests: > https://bugs.openjdk.java.net/browse/JDK-8158621 > <https://bugs.openjdk.java.net/browse/JDK-8158621> > https://bugs.openjdk.java.net/browse/JDK-8158485 > <https://bugs.openjdk.java.net/browse/JDK-8158485> > https://bugs.openjdk.java.net/browse/JDK-8158501 > <https://bugs.openjdk.java.net/browse/JDK-8158501> > > Jtreg tests: > https://bugs.openjdk.java.net/browse/JDK-8158389 > <https://bugs.openjdk.java.net/browse/JDK-8158389> > https://bugs.openjdk.java.net/browse/JDK-8158526 > <https://bugs.openjdk.java.net/browse/JDK-8158526> > https://bugs.openjdk.java.net/browse/JDK-8158496 > <https://bugs.openjdk.java.net/browse/JDK-8158496> > https://bugs.openjdk.java.net/browse/JDK-8158362 > <https://bugs.openjdk.java.net/browse/JDK-8158362> > https://bugs.openjdk.java.net/browse/JDK-8158512 > <https://bugs.openjdk.java.net/browse/JDK-8158512> > https://bugs.openjdk.java.net/browse/JDK-8156460 > <https://bugs.openjdk.java.net/browse/JDK-8156460> > https://bugs.openjdk.java.net/browse/JDK-8158377 > <https://bugs.openjdk.java.net/browse/JDK-8158377> > > Reason of failure: > The modifier value calculation was wrong. > > Note that with this fix the test > /java/awt/keyboard/AllKeyCode/AllKeyCode.java will fail due to the reason > that pressing number (0 to 9) after pressing arrow keys( up, down, left and > right) will generate corresponding Numpad keys code for number keys (0 to 9). > Whereas if the arrow key are pressed after number keys are pressed then there > is no problem. An issue will be created for this issue once this fix is > accepted. > > Thanks, > Manajit