[7u4] Request for review: 7124537 [macosx] Menu shortcuts for all menu items should be disabled if a menu itself is disabled
sergey.bylokhov at ORACLE.COM
Wed Feb 29 07:50:40 PST 2012
29.02.2012 19:06, Anthony Petrov wrote:
> 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?
There is a changes in formatting
> best regards,
> On 2/29/2012 4:50 PM, Sergey Bylokhov wrote:
>> Does anybody has a chance to review it?
>> 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:
Best regards, Sergey.
More information about the macosx-port-dev