<Swing Dev> [9] Review request for 8072900: Mouse events are captured by the wrong menu in OS X
Anton Nashatyrev
anton.nashatyrev at oracle.com
Thu Mar 5 17:19:29 UTC 2015
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.
>
>
More information about the swing-dev
mailing list