Looks fine.

> 
> Hello,
> 
> please review the fix
> 
> http://cr.openjdk.java.net/~azvegint/jdk/9/8177919/00/
> 
> for the issue
> 
> https://bugs.openjdk.java.net/browse/JDK-8177919
> 
> This fix removes throwing of ISE, this allows to use default menu bar with 
> LaF's other than Aqua (with apple.laf.useScreenMenuBar set to true).
> 
> This became possible after JDK-8166683[0] fix.
> 
> Current documentation of Desktop.setDefaultMenuBar() has implnotes:
> 
>     * @implNote Aqua Look and Feel should be active to support this on Mac OS.
> 
> I leave it unchanged, since I don't want to advertise the 
> apple.laf.useScreenMenuBar property.
> 
> [0] https://bugs.openjdk.java.net/browse/JDK-8166683
> 
> 
> -- 
> Thanks,
> Alexander.
> 

Reply via email to