Hi, Anton.
Thanks! The fix looks good.
On 05.03.2015 20:19, Anton Nashatyrev wrote:
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.
--
Best regards, Sergey.