Bug 388516 - Regression: Never ending exceptions after closing allv20130124-165836

perspectives and restarting

2 files changed, 14 insertions, 3 deletions

diff --git a/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/HandledContributionItem.java b/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/HandledContributionItem.java
index 4cf7674..fea0915 100644
--- a/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/HandledContributionItem.java
+++ b/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/HandledContributionItem.java
@@ -118,6 +118,7 @@ public class HandledContributionItem extends ContributionItem {
 
 		List<HandledContributionItem> itemsToCheck = new ArrayList<HandledContributionItem>();
 		List<Runnable> windowRunnables = new ArrayList<Runnable>();
+		final List<HandledContributionItem> orphanedToolItems = new ArrayList<HandledContributionItem>();
 
 		public void addWindowRunnable(Runnable r) {
 			windowRunnables.add(r);
@@ -142,8 +143,18 @@ public class HandledContributionItem extends ContributionItem {
 		}
 
 		public void run() {
-			for (HandledContributionItem hci : itemsToCheck) {
-				hci.updateItemEnablement();
+
+			for (final HandledContributionItem hci : itemsToCheck) {
+				// HACK. Remove orphaned entries. See bug 388516.
+				if (hci.model != null && hci.model.getParent() != null) {
+					hci.updateItemEnablement();
+				} else {
+					orphanedToolItems.add(hci);
+				}
+			}
+			if (!orphanedToolItems.isEmpty()) {
+				itemsToCheck.removeAll(orphanedToolItems);
+				orphanedToolItems.clear();
 			}
 
 			Runnable[] array = new Runnable[windowRunnables.size()];
diff --git a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/quickaccess/SearchField.java b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/quickaccess/SearchField.java
index 83437da..9911553 100644
--- a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/quickaccess/SearchField.java
+++ b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/quickaccess/SearchField.java
@@ -118,7 +118,7 @@ public class SearchField {
 			}
 
 			private void closeDropDown() {
-				if (shell.isDisposed() || text.isDisposed() || !shell.isVisible())
+				if (shell == null || shell.isDisposed() || text.isDisposed() || !shell.isVisible())
 					return;
 
 				quickAccessContents.doClose();