Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 324730 - CleanupAddon causes parts to be unrendered while performing DND operations
Summary: CleanupAddon causes parts to be unrendered while performing DND operations
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 1.0   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-08 08:21 EDT by Remy Suen CLA
Modified: 2010-09-17 16:21 EDT (History)
0 users

See Also:

Attachments
Re-order the operations so that the part is already in the new parent stack *before* blowing away th eold one (894 bytes, patch)
2010-09-17 16:18 EDT, Eric Moffatt CLA 		no flags Details | Diff
Re-order the operations so that the part is already in the new parent stack *before* blowing away the old (1.20 KB, patch)
2010-09-17 16:19 EDT, Eric Moffatt CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Remy Suen CLA 2010-09-08 08:21:20 EDT 
1. Move the 'Package Explorer' to the right below the 'Outline' view so it spawns a new stack.
2. Now drag it and move it to the same stack as the 'Outline'.
3. It will get disposed although it may not be obvious from looking at the part.

Thread [main] (Suspended (breakpoint at line 488 in PartImpl))	
	PartImpl.setWidget(Object) line: 488	
	ContributedPartRenderer(SWTPartRenderer).unbindWidget(MUIElement) line: 112	
	ContributedPartRenderer(SWTPartRenderer).disposeWidget(MUIElement) line: 131	
	PartRenderingEngine.removeGui(MUIElement) line: 532	
	ElementReferenceRenderer.disposeWidget(MUIElement) line: 108	
	PartRenderingEngine.removeGui(MUIElement) line: 532	
	PartRenderingEngine.removeGui(MUIElement) line: 528	
	PartRenderingEngine.removeGui(MUIElement) line: 528	
	PartRenderingEngine$1.handleEvent(Event) line: 128	
	UIEventHandler.handleEvent(Event) line: 41	
	EventHandlerWrapper.handleEvent(Event, Permission) line: 188	
	EventHandlerTracker.dispatchEvent(Object, Object, int, Object) line: 198	
	EventManager.dispatchEvent(Set, EventDispatcher, int, Object) line: 227	
	ListenerQueue.dispatchEventSynchronous(int, Object) line: 149	
	EventAdminImpl.dispatchEvent(Event, boolean) line: 139	
	EventAdminImpl.sendEvent(Event) line: 78	
	EventComponent.sendEvent(Event) line: 39	
	EventBroker.send(String, Object) line: 73	
	UIEventPublisher.notifyChanged(Notification) line: 58	
	PartSashContainerImpl(BasicNotifierImpl).eNotify(Notification) line: 380	
	PartSashContainerImpl(UIElementImpl).setToBeRendered(boolean) line: 267	
	CleanupAddon$1$1.run() line: 108	
	RunnableLock.run() line: 35	
	UISynchronizer(Synchronizer).runAsyncMessages(boolean) line: 134
Comment 1 Eric Moffatt CLA 2010-09-17 16:18:30 EDT 
Created attachment 179149 [details]
Re-order the operations so that the part is already in the new parent stack *before* blowing away th eold one
Comment 2 Eric Moffatt CLA 2010-09-17 16:19:52 EDT 
Created attachment 179150 [details]
Re-order the operations so that the part is already in the new parent stack *before* blowing away the old
Comment 3 Eric Moffatt CLA 2010-09-17 16:20:49 EDT 
Committed in >20100917. Applied the patch.
Comment 4 Eric Moffatt CLA 2010-09-17 16:21:18 EDT 
Marking this one as FIXED.