Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 330130 - CCE updating Scripts node
Summary: CCE updating Scripts node
Status: RESOLVED FIXED
Alias: None
Product: JSDT
Classification: WebTools
Component: Debug (show other bugs)
Version: 3.3   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.3 M4   Edit
Assignee: Michael Rennie CLA
QA Contact: Simon Kaegi CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-12 14:23 EST by Michael Rennie CLA
Modified: 2010-11-12 14:30 EST (History)
1 user (show)

See Also:


Attachments
fix (3.34 KB, patch)
2010-11-12 14:23 EST, Michael Rennie CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Rennie CLA 2010-11-12 14:23:28 EST
Created attachment 183032 [details]
fix

code from HEAD

If an event is sent to update the Scripts node in the debug view we incorrectly delegate to the parent event handler causing a ClassCastException. We should be checking the source of the event and redirecting it to the Scripts node handling code as needed.

java.lang.ClassCastException: org.eclipse.wst.jsdt.debug.internal.core.model.ScriptGroup cannot be cast to org.eclipse.debug.core.model.IDebugTarget
	at org.eclipse.debug.internal.ui.viewers.update.DebugTargetEventHandler.handleChange(DebugTargetEventHandler.java:47)
	at org.eclipse.debug.internal.ui.viewers.update.EventHandlerModelProxy.dispatchChange(EventHandlerModelProxy.java:238)
	at org.eclipse.debug.internal.ui.viewers.update.EventHandlerModelProxy.handleDebugEvents(EventHandlerModelProxy.java:141)
	at org.eclipse.debug.core.DebugPlugin$EventNotifier.run(DebugPlugin.java:1101)
	at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:42)
	at org.eclipse.debug.core.DebugPlugin$EventNotifier.dispatch(DebugPlugin.java:1135)
	at org.eclipse.debug.core.DebugPlugin$EventDispatchJob.run(DebugPlugin.java:399)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:54)
Comment 1 Michael Rennie CLA 2010-11-12 14:30:17 EST
applied to HEAD