<Swing Dev> [9] Review request for 8072900: Mouse events are captured by the wrong menu in OS X

[Alexander Potochkin]

Hello Anton

Looks good!

Thanks
alexp

On 3/5/2015 8:19 PM, 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.
>>
>>
>