[7u4] Request for review: 7124537 [macosx] Menu shortcuts for all menu items should be disabled if a menu itself is disabled

[Anthony Petrov]

Hi Sergey,

I'm not an expert in the Menu code, but generally the fix looks good. By 
the way, LWCToolkit.java doesn't contain any changes. Should it?

--
best regards,
Anthony

On 2/29/2012 4:50 PM, Sergey Bylokhov wrote:
> Does anybody has a chance to review it?
> Thanks.
> 
> 22.02.2012 22:08, Sergey Bylokhov wrote:
>> Hi Everyone,
>> Change description:
>> 1 CMenu.java: state of all elements for the current menu changes now. 
>> Before the fix, menu in menubar didn't change status.
>> 2 CMenuItem.java: menu item can be enabled only if parent is enabled 
>> too. Native code execute only if status was changed.
>> Fix was created on the java level instead of native. Please comment if 
>> you have an objections.
>>
>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7124537
>> Webrev can be found at: 
>> http://cr.openjdk.java.net/~serb/7124537/webrev.00/
>>
> 
>