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

Bug 347014

Summary: Leak of N number of listeners for N tab changes, preserved until the folder is destroyed
Product: [Eclipse Project] e4 Reporter: Remy Suen <remy.suen>
Component: UIAssignee: Remy Suen <remy.suen>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P3    
Version: 1.0   
Target Milestone: 4.1 RC3   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
StackRenderer patch v1 none

Description Remy Suen CLA 2011-05-24 11:52:11 EDT 
A MenuDetectListener is attached to the tab folder in StackRenderer's showTab(MUIElement) implementation. Since this method is called every time there's a tab change, we are creating N listeners for N number of tab changes.
Comment 1 Remy Suen CLA 2011-05-24 12:04:09 EDT 
Created attachment 196460 [details]
StackRenderer patch v1

Change the renderer to hook the listener in its hookControllerLogic(MUIElement) method.
Comment 2 Remy Suen CLA 2011-05-24 12:04:46 EDT 
(In reply to comment #1)
> Created attachment 196460 [details]
> StackRenderer patch v1

Patch released to CVS HEAD.