<AWT Dev> [7u10] Review request for 7186371: [macosx] Main menu shortcuts not displayed (7u6 regression)

[Artem Ananiev]

The fix looks fine. Please, add a comment to SMI.setLabel() and 
SMI.setShortcut() with explanation that we don't call to super methods 
from there.

Thanks,

Artem

On 9/5/2012 2:03 PM, Sergey Bylokhov wrote:
> Hi Everyone,
> Please review the fix. This is backport from jdk8 to 7u10
> When we translate calls from our swing menu components to awt peer we
> resets information about shortcut in the setLabel().
> This happens because of ScreenMenuItem.setAccelerator() method call
> peers setLabel(..,..,..) directly and does not initialize
> ScreenMenuItem.shortcut property.
> But default implementation of ScreenMenuItem.setLabel() assumes that
> this field(shortcut) will be initialized.
>
> This works on jdk6 because it does not reset shortcut if null or empty
> shortcut is provided.
> As a solution we can use peers methods directly in both cases.
>
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7186371
> Webrev can be found at: http://cr.openjdk.java.net/~serb/7186371/webrev.00
> jdk8 changeset: http://hg.openjdk.java.net/jdk8/awt/jdk/rev/f54660c18774
>