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

Bug 333493

Summary:

disposed widget exception when closing editor with Browser with Ctrl+F4

Product:

[Eclipse Project] Platform

Reporter:

Grant Gayed <grant_gayed>

Component:

SWT

Assignee:

Grant Gayed <grant_gayed>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

Silenio_Quarti

Version:

3.6.1

Flags:

Silenio_Quarti: review+

Target Milestone:

3.6.2

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
patch	none

Description Grant Gayed

2011-01-04 11:23:34 EST

- open an .html file in eclipse's Web Browser editor
- close the editor with Ctrl+F4
- the exception below occurs:

org.eclipse.swt.SWTException: Widget is disposed
at org.eclipse.swt.SWT.error(SWT.java:4083)
at org.eclipse.swt.SWT.error(SWT.java:3998)
at org.eclipse.swt.SWT.error(SWT.java:3969)
at org.eclipse.swt.widgets.Widget.error(Widget.java:468)
at org.eclipse.swt.widgets.Widget.checkWidget(Widget.java:340)
at org.eclipse.swt.widgets.Widget.getData(Widget.java:554)
at org.eclipse.swt.ole.win32.OleFrame.getMsgProc(OleFrame.java:224)
at org.eclipse.swt.internal.win32.OS.PeekMessageW(Native Method)
at org.eclipse.swt.internal.win32.OS.PeekMessage(OS.java:3024)
at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3652)
at org.eclipse.ui.internal.Workbench.runEventLoop(Workbench.java:2640)
at org.eclipse.ui.internal.Workbench.runUI(Workbench.java:2604)
at org.eclipse.ui.internal.Workbench.access$4(Workbench.java:2438)
at org.eclipse.ui.internal.Workbench$7.run(Workbench.java:671)
at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:332)
at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:664)
at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:149)
at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:115)
at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:110)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:79)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:369)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:179)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:619)
at org.eclipse.equinox.launcher.Main.basicRun(Main.java:574)
at org.eclipse.equinox.launcher.Main.run(Main.java:1407)

Comment 1 Grant Gayed

2011-01-04 12:50:01 EST

SSQ can you review the patch in the next comment for inclusion in 3.6.2?  The only part I'm not sure about is whether just returning 0 is the correct thing to do in the disposed cases.

Comment 2 Grant Gayed

2011-01-04 12:51:42 EST

Created attachment 186031 [details]
patch

Comment 3 Grant Gayed

2011-01-04 14:43:03 EST

fixed in the 3.6.2 and 3.7 streams > 20110104