Hi Sergey, I ran the relevant test cases: ActionListenerCalledTwiceTest ScreenMenuBarInputTwice with different keyboard preferences and locales. Also tried different LAFs but found that only default (aqua) LAF is relevant for this case.
With Regards, Avik Niyogi > On 31-May-2016, at 7:05 pm, Sergey Bylokhov <sergey.bylok...@oracle.com> > wrote: > > Hi, Avik. > Please confirm that you run all tests for the previous regressions which were > fixed in this code recently. (use different system locales). > > On 31.05.16 16:28, Avik Niyogi wrote: >> Hi All, >> >> Kindly review the fix for JDK9. >> *Bug: *https://bugs.openjdk.java.net/browse/JDK-8158230 >> >> *Webrev:* http://cr.openjdk.java.net/~aniyogi/8158230/webrev.00/ >> >> *Issue*: ActionEvent is not fired for menu item with option >> apple.laf.useScreenMenuBar >> >> *Cause*: The case for event that is not a Keydown has to be handled >> >> *Fix*: With fixing the if - else loop, the correct behaviour was confirmed. >> >> With Regards, >> Avik Niyogi > > > -- > Best regards, Sergey.
