Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 329711

Summary: [Progress] AccessibleListener leaked in ProgressAnimationItem
Product: [Eclipse Project] Platform Reporter: Nick Edgar <n.a.edgar>
Component: UIAssignee: Platform UI Triaged <platform-ui-triaged>
Status: CLOSED DUPLICATE QA Contact: Prakash Rangaraj <prakash>
Severity: normal    
Priority: P3 CC: daniel_megert
Version: 3.6   
Target Milestone: 3.7   
Hardware: All   
OS: All   
Whiteboard:

Description Nick Edgar CLA 2010-11-08 16:31:33 EST
In ProgressAnimationItem.initButton, it does:

    	toolbar.getAccessible().addAccessibleListener(new AccessibleAdapter() {
        	public void getName(AccessibleEvent e) {
        		e.result = tt;
        	}
        });

initButton can be called many times over the life of the item, though, so a new listener gets added each time.

One possible fix is to add the listener when the toolbar is created, and use toolbar.get/setData() to carry the name.
Comment 1 Prakash Rangaraj CLA 2010-11-09 00:44:54 EST
Sounds like dup of Bug#315620, but the patch doesn't contain the fix for this one.
Comment 2 Dani Megert CLA 2010-11-09 02:58:16 EST
(In reply to comment #1)
> Sounds like dup of Bug#315620,
Agreed.

> but the patch doesn't contain the fix for this one.
The patch fixed things at a higher level.

Nick, can you please try 3.6.1 or newer?
Comment 3 Prakash Rangaraj CLA 2010-11-12 01:14:11 EST
(In reply to comment #2)
> (In reply to comment #1)
> > Sounds like dup of Bug#315620,
> Agreed.
> 
> > but the patch doesn't contain the fix for this one.
> The patch fixed things at a higher level.
> 
> Nick, can you please try 3.6.1 or newer?

Nick, 

    I'm closing this bug. Please try it on 3.6.1 and if its reproducible, reopen the original bug (Bug# 315620) and add your comments.

*** This bug has been marked as a duplicate of bug 315620 ***