Hello, AWT Team. Please review the new version of the fix: http://cr.openjdk.java.net/~pchelko/8024329/webrev.01/
This version of the fix does not expose the package-private class com.apple.laf.ScreenMenuItemCheckbox Instead we override the setState method to no-op and force the setState only when it's really needed. With best regards. Petr. On 10.10.2013, at 18:19, Petr Pchelko <[email protected]> wrote: > Hello, AWT Team. > > Please review the fix for the issue: > https://bugs.openjdk.java.net/browse/JDK-8024329 > The fix is available at: > http://cr.openjdk.java.net/~pchelko/8024329/webrev.00/ > > The problem: for AWT menu the peer should update the menu item state > unconditionally. However for the Swing menu items used in screen menubar > swing should decide wether to update the state or not. So we should not > unconditionally update the sate. > There's no regression test as there's no way to understand if the native menu > item is checked or not. > > With best regards. Petr.
