Revert "Bug 402429 - hiddenToolBarItem perspective extension does not react to"

This reverts commit 4ab730aaad924f28aef63a3f4650c797be9fa0f6.

1 files changed, 5 insertions, 19 deletions

diff --git a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/CoolBarToTrimManager.java b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/CoolBarToTrimManager.java
index 43063c2f15e..632db2b0a1d 100644
--- a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/CoolBarToTrimManager.java
+++ b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/CoolBarToTrimManager.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2011, 2013 IBM Corporation and others.
+ * Copyright (c) 2011, 2012 IBM Corporation and others.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -704,10 +704,12 @@ public class CoolBarToTrimManager extends ContributionManager implements ICoolBa
 				container, null);
 
 		for (IContributionItem item : manager.getItems()) {
-			if (item == null || !isChildVisible(item)) {
+			if (item == null) {
+				continue;
+			}
+			if (renderer.getToolElement(item) != null) {
 				continue;
 			}
-
 			if (item instanceof IToolBarContributionItem) {
 				IToolBarManager manager2 = ((IToolBarContributionItem) item).getToolBarManager();
 				//new Exception("fill(MToolBar container, IContributionManager manager) with " //$NON-NLS-1$
@@ -736,20 +738,4 @@ public class CoolBarToTrimManager extends ContributionManager implements ICoolBa
 			}
 		}
 	}
-
-	private boolean isChildVisible(IContributionItem item) {
-		Boolean v;
-
-		IContributionManagerOverrides overrides = getOverrides();
-		if (overrides == null) {
-			v = null;
-		} else {
-			v = getOverrides().getVisible(item);
-		}
-
-		if (v != null) {
-			return v.booleanValue();
-		}
-		return item.isVisible();
-	}
 }