Thanks.
Looks fine.

On 31.05.16 16:38, Avik Niyogi wrote:
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.



--
Best regards, Sergey.

Reply via email to