<AWT Dev> [9] Review Request: JDK-8007220 [macosx] Setting popupmenu on TrayIcon do not work if done *after* adding TrayIcon

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Tue Dec 17 02:52:41 PST 2013


On 12/16/13 5:25 PM, Petr Pchelko wrote:
> Hello, Sergey.
>
>> Since you add new call to addNotify(), can you check that we call removeNotify when necessary?
> On Win and Linux we were calling it properly for popups, but on mac os x we haven't been doing that. The new version of the fix fixes that.
Thanks for this check. Is it possible to create a new test for this issue?
>
> The new version is here:
> http://cr.openjdk.java.net/~pchelko/9/8007220/webrev.01/
>
> With best regards. Petr.
>
> On 16.12.2013, at 16:38, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>
>> Hi, Petr.
>> Since you add new call to addNotify(), can you check that we call removeNotify when necessary?
>>
>> On 16.12.2013 16:02, Petr Pchelko wrote:
>>> Hello, AWT Team.
>>>
>>> Please review the fix for the issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8007220
>>> The fix is available at:
>>> http://cr.openjdk.java.net/~pchelko/9/8007220/webrev/
>>>
>>> The problem: the CTrayIcon.popup field was set only in a constructor, but it should be updated every time the popup is opened.
>>>
>>> With best regards. Petr.
>>
>> -- 
>> Best regards, Sergey.
>>


-- 
Best regards, Sergey.



More information about the awt-dev mailing list