[8] Review request for 7186371:  [macosx] Main menu shortcuts not	displayed (7u6 regression)
    Sergey Bylokhov 
    Sergey.Bylokhov at oracle.com
       
    Tue Aug  7 04:39:39 PDT 2012
    
    
  
Hi Everyone,
Please review the fix.
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
-- 
Best regards, Sergey.
    
    
More information about the macosx-port-dev
mailing list