<AWT Dev> [8] Review request for 7160951: [macosx] ActionListener called twice for JMenuItem using ScreenMenuBar

[Sergey Bylokhov]

In this case it would be good to check it in another CR. Fix itself 
looks good.

07.09.2012 23:55, Leonid Romanov wrote:
> 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 <mailto: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 called 
>>> 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/ 
>>> <http://cr.openjdk.java.net/%7Eleonidr/7160951/webrev.01/>
>>>
>>> Thanks,
>>> Leonid.
>>
>>
>> -- 
>> Best regards, Sergey.
>


-- 
Best regards, Sergey.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20120908/be336445/attachment.html