<AWT Dev>  Review request for CR 7079254 Toolkit eventListener leaks memory
petr.pchelko at oracle.com
Mon Dec 24 23:34:18 PST 2012
Thank you for your feedback.
Here is the new version of the fix:
Changes from the previous one:
1. Added a test.
2. Moved creadLightweightDispatcherOnRemove into a synchronized section.
On Dec 24, 2012, at 4:03 PM, Artem Ananiev wrote:
> Hi, Petr,
> the fix looks fine. Please, move clearLightweightDispatcherOnRemove() into the synchronized section, as lightweight dispatcher is used under tree lock.
> As Sergey suggested, it would be fine to have a regression test for this change.
> On 12/21/2012 4:50 PM, Petr Pchelko wrote:
>> Could you please review the fix for the issue
>> 7079254 Toolkit eventListener leaks memory
>> The webrev is available at:
>> The LightweightDispatcher stored strong references to mouseEventTarget and targetLastEntered which were not cleaned up when the component is removed, which lead to a memory leak. The isCleaned field is added to be able to consume events which would be dispatched to a removed component if we did not clean up a reference to it.
>> The fix testet on Windows and Mac on toy apps and automatic regression tests related to event dispatching, Component, Container, dnd and Mouse
>> Best, Petr.
More information about the awt-dev