Hi, Petr.
The fix looks good. But please add a dot at the and of the description isEmbedded(), before @return tag.

On 02.07.2014 18:25, Petr Pchelko wrote:
Hello, AWT Team.

Please review the fix for the issue:
https://bugs.openjdk.java.net/browse/JDK-8048549
The fix is available at:
http://cr.openjdk.java.net/~pchelko/9/8048549/webrev/

We need to disable the screenMenuBar if AWT is embedded into FX. Only the 
top-level UI toolkit should work with the global menu bar.
We are already doing the same thing in FX. I've also added some cleanup into 
the fix. No test provided because we do not have tests for
embedded mode.

With best regards. Petr.


--
Best regards, Sergey.

Reply via email to