[8] Review request for 7160951: [macosx] ActionListener called twice for JMenuItem using ScreenMenuBar

[Leonid Romanov]

I dunno. I just did some cleanup of existing code, I didn't dig deeper.  I'll investigate it. 

On Sep 7, 2012, at 11:53 PM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:

> Hi, Leonid.
> Why we cannot use javaExtMask everywhere in CMenuItem.java.setLabel(...) && CMenuItem.m ?
> 
> 07.09.2012 22:41, Leonid Romanov wrote:
>> Hi,
>> Please review a fix for 7160951: [macosx] ActionListener calle d twice for JMenuItem using ScreenMenuBar.  Although a number of files have been changed, only CMenuItem.m changes constitutes the fix, the rest of the changes is a bit of code cleanup.
>> The fix has been contributed by Marco Dinacci. Thanks!
>> 
>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7160951
>> Webrev: http://cr.openjdk.java.net/~leonidr/7160951/webrev.01/
>> 
>> Thanks,
>> Leonid.
> 
> 
> -- 
> Best regards, Sergey.