Hi Sergey,
thanks for review!
I've fixed the test as you suggested (renamed, using Robot, test
for all LAFs). Please take a look ta the new version:
http://cr.openjdk.java.net/~anashaty/8072900/9/webrev.01/
<http://cr.openjdk.java.net/%7Eanashaty/8072900/9/webrev.01/>
Thanks!
Anton.
On 05.03.2015 19:21, Sergey Bylokhov wrote:
Hi, Anton.
The fix looks fine, but please use Robot.waitForIdle() instead of
realSync. Because SunToolkit will not be available for such tests in
jdk9 soon. I suppose test name can be changed too, but it is not
important.
On 04.03.2015 17:44, Anton Nashatyrev wrote:
Hello,
could you please review the following fix:
fix: http://cr.openjdk.java.net/~anashaty/8072900/9/webrev.00/
<http://cr.openjdk.java.net/%7Eanashaty/8072900/9/webrev.00/>
bug: https://bugs.openjdk.java.net/browse/JDK-8072900
The problem: On MacOS if there are two frames with menu, the first is
opened and the mouse moved over the second it causes the second menu
to be opened.
The reason: the AquaMenuUI creates its own MouseHandler and the logic
of mouseEnter() has not been completely ported from the BasicMenuUI.
Thanks!
Anton.