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

Bug 446697

Summary:	Leaking inputManager listeners in new Hover code
Product:	[ECD] Orion	Reporter:	Eric Moffatt <emoffatt>
Component:	Client	Assignee:	Project Inbox <orion.client-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	unspecified
Target Milestone:	7.0
Hardware:	PC
OS:	Windows 7
Whiteboard:

Description Eric Moffatt

2014-10-10 14:27:28 EDT

Because of a misunderstanding of the lifecycle of the inputManager / Hover instance we were adding multiple input changed listeners to the inputManager.

The solution is to move the listener into the HoverFactory, do the filtering there and provide the actual Hover instance with access to the factory so it can get the current list of filtered service providers.

Comment 1 Eric Moffatt

2014-10-10 14:40:53 EDT

I've pushed a Gerrit for review:

https://git.eclipse.org/r/#/c/34739/

Comment 2 Eric Moffatt

2014-10-14 15:00:36 EDT

Committed by Silenio...thanks !!