Summary: | Widget is Disposed error after Cancelling a DStore Shell | ||||||
---|---|---|---|---|---|---|---|
Product: | [Tools] Target Management | Reporter: | Kevin Doyle <kjdoyle> | ||||
Component: | RSE | Assignee: | David McKnight <dmcknigh> | ||||
Status: | CLOSED FIXED | QA Contact: | Martin Oberhuber <mober.at+eclipse> | ||||
Severity: | minor | ||||||
Priority: | P3 | CC: | dmcknigh | ||||
Version: | 2.0 | Flags: | mober.at+eclipse:
review+
|
||||
Target Milestone: | 2.0.1 | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Kevin Doyle
2007-09-13 13:40:59 EDT
Reproduced on Linux easily. It should be possible to find the issue by placing a breakpoint at SystemView#updatePropertySheet (line 5503). I consider this minor though since there is no harm except logging the exception. Created attachment 78894 [details]
patch to use context string instead of context when updating the property sheet
I've attached a patch to use the context string instead of the context object when updating the property sheet for a shell. If getContext() were used, then we would end up doing a query on the main thread. Because this is on the main thread, the Display.readAndDispatch() gets called while waiting for the query response. Read and dispatch ends up allowing some refresh events through via the system view and that seems to be what's causing the widget disposed errors.
Martin, could you review the patch? It's a very small change and fairly low-risk. Not sure whether this should wait for 3.0 or go in 2.0.1. Patch looks good. I'm fine with putting this in 2.0.1 - feel free to check in and I'll retag it. Okay, I've committed the fix to cvs. Verified fixed in I20071108. |