Project: Eclipse_Platform_UI
Bug 223074 [ErrorHandling] WorkbenchStatusDialog API review
Reported: 2008-03-18 09:27:26 | Status: verified fixed

As M6 is the API freeze I think we need to decide if some changes are required here 1) It is not a dialog it is something that manages the opening of a dialog. It needs to be renamed. 2) Test mode just means "run in sync exec". I am not sure why we even want this as an asyncExec and a syncExec is a no-op on the UI Thread. 3) Is it the intention of this class to just provide an API way to invoke an internal dialog? The dialog itself can't be modified as InternalDialog is where all of the work is done.

Files changed in fix:
bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/statushandlers/DefaultDetailsArea.java bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/statushandlers/StackTraceSupportArea.java bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/statushandlers/AbstractStatusAreaProvider.java bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/statushandlers/WorkbenchErrorHandler.java bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/statushandlers/WorkbenchStatusDialog.java bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/statushandlers/WorkbenchStatusDialogManager.java