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

Bug 22872

Summary: Remove All Terminated causes dancing scrollbar
Product: [Eclipse Project] Platform Reporter: Peter Burka <peter_burka>
Component: DebugAssignee: Darin Wright <darin.eclipse>
Status: RESOLVED DUPLICATE QA Contact:
Severity: enhancement    
Priority: P3 CC: jared_burns, jed.anderson
Version: 2.0   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Peter Burka CLA 2002-08-26 17:35:41 EDT
When you have several terminated launches in the Debug view, and 
choose 'Remove All Terminated', the scrollbar starts dancing around wildly for 
a few moments.

It looks like the thumb going from top to bottom and back again several times, 
continually growing as it does so. The visual effect is actually quite 
interesting the first couple of times you see it.
Comment 1 Darin Wright CLA 2002-08-26 21:54:06 EDT
Cool. And you'd like us to fix it?
Comment 2 Jared Burns CLA 2002-08-27 10:44:07 EDT
The problem is that we remove the launches from the viewer one at a time. 
This is because the LaunchManager's API only allows us to remove one launch 
at a time. We could add a method to remove multiple launches at once, 
because ILaunchManager is "not intended to be implemented by clients" but 
this wouldn't help us because the ILaunchListener API, which *is* intended 
to be implemented by clients only takes a single launch.

I think the only way to work around this bug would be to add a new 
listener, IMultipleLaunchListener, which extends ILaunchListener and adds 
methods for multiple-launch notifications. We'd have to still support the 
old ILaunchListener, but we could deprecate it at migrate our code to the 
new listener.

This would allow our UI to remove all of the launches in one operation.
Comment 3 Darin Wright CLA 2002-08-27 14:07:58 EDT
To avoid new API, we should investigate the action unhooking the view as a 
launch listener, removing the lanuches, and then re-hooking and refreshing the 
view.
Comment 4 Darin Wright CLA 2002-08-27 14:49:12 EDT
*** Bug 22907 has been marked as a duplicate of this bug. ***
Comment 5 Jed Anderson CLA 2002-08-27 14:58:22 EDT
I believe the correct long term solution is introduce the new interface.  This also means adding 
new methods to the launch manager.

I am worried that there are other interfaces in the debug 
API that should take arrays of elements rather than single elements.
Comment 6 Darin Wright CLA 2002-08-27 15:06:58 EDT
The breakpoint, launch, and expression listener interfaces are all singular.
Comment 7 Darin Wright CLA 2002-10-16 12:34:37 EDT
This has been fixed.

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