Attachment 181191 Details for Bug 325392 – MenuManagerRenderer v02

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

[patch] MenuManagerRenderer v02

menuOnly-v02.txt (text/plain), 74.90 KB, created by Paul Webster

on 2010-10-19 11:23:37 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Paul Webster

Created: 2010-10-19 11:23:37 EDT

Size: 74.90 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.e4.ui.tests
>Index: UIAllTests.launch
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/tests/org.eclipse.e4.ui.tests/UIAllTests.launch,v
>retrieving revision 1.6
>diff -u -r1.6 UIAllTests.launch
>--- UIAllTests.launch	19 Jul 2010 15:22:51 -0000	1.6
>+++ UIAllTests.launch	19 Oct 2010 15:22:31 -0000
>@@ -27,7 +27,7 @@
> <booleanAttribute key="org.eclipse.jdt.junit.KEEPRUNNING_ATTR" value="false"/>
> <stringAttribute key="org.eclipse.jdt.junit.TESTNAME" value=""/>
> <stringAttribute key="org.eclipse.jdt.junit.TEST_KIND" value="org.eclipse.jdt.junit.loader.junit3"/>
>-<stringAttribute key="org.eclipse.jdt.launching.JRE_CONTAINER" value="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/J2SE-1.5"/>
>+<stringAttribute key="org.eclipse.jdt.launching.JRE_CONTAINER" value="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.6"/>
> <stringAttribute key="org.eclipse.jdt.launching.MAIN_TYPE" value="org.eclipse.e4.ui.tests.UIAllTests"/>
> <stringAttribute key="org.eclipse.jdt.launching.PROGRAM_ARGUMENTS" value="-os ${target.os} -ws ${target.ws} -arch ${target.arch} -nl ${target.nl}"/>
> <stringAttribute key="org.eclipse.jdt.launching.PROJECT_ATTR" value="org.eclipse.e4.ui.tests"/>
>Index: src/org/eclipse/e4/ui/tests/workbench/MMenuItemTest.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/tests/org.eclipse.e4.ui.tests/src/org/eclipse/e4/ui/tests/workbench/MMenuItemTest.java,v
>retrieving revision 1.9
>diff -u -r1.9 MMenuItemTest.java
>--- src/org/eclipse/e4/ui/tests/workbench/MMenuItemTest.java	14 Oct 2010 18:21:17 -0000	1.9
>+++ src/org/eclipse/e4/ui/tests/workbench/MMenuItemTest.java	19 Oct 2010 15:22:31 -0000
>@@ -34,11 +34,15 @@
> import org.eclipse.e4.ui.model.application.ui.menu.MMenuItem;
> import org.eclipse.e4.ui.model.application.ui.menu.MMenuSeparator;
> import org.eclipse.e4.ui.model.application.ui.menu.impl.MenuFactoryImpl;
>-import org.eclipse.e4.ui.workbench.renderers.swt.MenuRenderer;
>+import org.eclipse.e4.ui.workbench.renderers.swt.MenuManagerRenderer;
> import org.eclipse.e4.ui.workbench.swt.factories.IRendererFactory;
>+import org.eclipse.jface.action.IContributionItem;
>+import org.eclipse.jface.action.MenuManager;
> import org.eclipse.swt.SWT;
> import org.eclipse.swt.widgets.Event;
>+import org.eclipse.swt.widgets.Menu;
> import org.eclipse.swt.widgets.MenuItem;
>+import org.eclipse.swt.widgets.Widget;
> 
> public class MMenuItemTest extends TestCase {
> 	protected IEclipseContext appContext;
>@@ -82,6 +86,8 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>+		((MenuManager) ((Widget) menu.getWidget()).getData()).updateAll(true);
>+
> 		Object widget = menuItem.getWidget();
> 		assertNotNull(widget);
> 		assertTrue(widget instanceof MenuItem);
>@@ -157,6 +163,8 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>+		((MenuManager) ((Widget) menu.getWidget()).getData()).updateAll(true);
>+
> 		Object widget1 = menuItem1.getWidget();
> 		assertNotNull(widget1);
> 		assertTrue(widget1 instanceof MenuItem);
>@@ -212,6 +220,8 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>+		((MenuManager) ((Widget) menu.getWidget()).getData()).updateAll(true);
>+
> 		Object widget1 = menuItem.getWidget();
> 		assertNotNull(widget1);
> 		assertTrue(widget1 instanceof MenuItem);
>@@ -228,7 +238,6 @@
> 		MCommand command = CommandsFactoryImpl.eINSTANCE.createCommand();
> 
> 		command.setElementId("commandId");
>-		command.setCommandName("CommandForTest");
> 
> 		menuItem.setCommand(command);
> 		menuItem.setType(ItemType.CHECK);
>@@ -240,13 +249,16 @@
> 		MApplication application = ApplicationFactoryImpl.eINSTANCE
> 				.createApplication();
> 		application.getChildren().add(window);
>-		application.getCommands().add(command);
> 		application.setContext(appContext);
> 		appContext.set(MApplication.class.getName(), application);
> 
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>+		MenuManager barManager = (MenuManager) ((Menu) menu.getWidget())
>+				.getData();
>+		barManager.updateAll(true);
>+
> 		Object widget1 = menuItem.getWidget();
> 		assertNotNull(widget1);
> 		assertTrue(widget1 instanceof MenuItem);
>@@ -255,7 +267,7 @@
> 		assertTrue(menuItemWidget.getSelection());
> 	}
> 
>-	public void XXXtestSubMenuCreation() throws Exception {
>+	public void testSubMenuCreation() throws Exception {
> 		MWindow window = BasicFactoryImpl.eINSTANCE.createWindow();
> 		MMenu menuBar = MenuFactoryImpl.eINSTANCE.createMenu();
> 		menuBar.setElementId("org.eclipse.ui.main.menu");
>@@ -289,9 +301,20 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>+		MenuManagerRenderer renderer = getRenderer(appContext, menuBar);
>+		MenuManager manager = renderer.getManager(menuBar);
>+		assertNotNull("failed to create menu bar manager", manager);
>+
>+		assertEquals(1, manager.getSize());
>+
>+		MenuManager fileManager = (MenuManager) manager.getItems()[0];
>+		MenuManager fileR = renderer.getManager(fileMenu);
>+		assertEquals(fileManager, fileR);
>+
>+		assertEquals(3, fileManager.getSize());
> 	}
> 
>-	public void XXXtestTbrItem() throws Exception {
>+	public void testTbrItem() throws Exception {
> 		MWindow window = BasicFactoryImpl.eINSTANCE.createWindow();
> 		MMenu menuBar = MenuFactoryImpl.eINSTANCE.createMenu();
> 		menuBar.setElementId("org.eclipse.ui.main.menu");
>@@ -326,11 +349,20 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>-		// MenuRenderer renderer = getRenderer(appContext, menuBar);
>+		MenuManagerRenderer renderer = getRenderer(appContext, menuBar);
>+		MenuManager manager = renderer.getManager(menuBar);
>+		assertNotNull("failed to create menu bar manager", manager);
>+
>+		assertEquals(1, manager.getSize());
>+
>+		MenuManager fileManager = (MenuManager) manager.getItems()[0];
>+		MenuManager fileR = renderer.getManager(fileMenu);
>+		assertEquals(fileManager, fileR);
> 
>+		assertEquals(2, fileManager.getSize());
> 	}
> 
>-	public void XXXtestInvisibleItem() throws Exception {
>+	public void testInvisibleItem() throws Exception {
> 		MWindow window = BasicFactoryImpl.eINSTANCE.createWindow();
> 		MMenu menuBar = MenuFactoryImpl.eINSTANCE.createMenu();
> 		menuBar.setElementId("org.eclipse.ui.main.menu");
>@@ -365,11 +397,22 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>-		// MenuRenderer renderer = getRenderer(appContext, menuBar);
>+		MenuManagerRenderer renderer = getRenderer(appContext, menuBar);
>+		MenuManager manager = renderer.getManager(menuBar);
>+		assertNotNull("failed to create menu bar manager", manager);
>+
>+		assertEquals(1, manager.getSize());
>+
>+		MenuManager fileManager = (MenuManager) manager.getItems()[0];
>+		MenuManager fileR = renderer.getManager(fileMenu);
>+		assertEquals(fileManager, fileR);
> 
>+		assertEquals(3, fileManager.getSize());
>+
>+		assertEquals(false, fileManager.getItems()[2].isVisible());
> 	}
> 
>-	public void XXXtestMenuContribution() throws Exception {
>+	public void testMenuContribution() throws Exception {
> 		MWindow window = BasicFactoryImpl.eINSTANCE.createWindow();
> 		MMenu menuBar = MenuFactoryImpl.eINSTANCE.createMenu();
> 		menuBar.setElementId("org.eclipse.ui.main.menu");
>@@ -404,9 +447,17 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>+		MenuManagerRenderer renderer = getRenderer(appContext, menuBar);
>+
>+		MenuManager fileManager = renderer.getManager(fileMenu);
>+		assertNotNull("No file menu?", fileManager);
>+
>+		assertEquals(4, fileManager.getSize());
>+
>+		assertEquals("mmc.item1", fileManager.getItems()[3].getId());
> 	}
> 
>-	public void XXXtestWithVisible() throws Exception {
>+	public void testWithVisible() throws Exception {
> 		MWindow window = BasicFactoryImpl.eINSTANCE.createWindow();
> 		MMenu menuBar = MenuFactoryImpl.eINSTANCE.createMenu();
> 		menuBar.setElementId("org.eclipse.ui.main.menu");
>@@ -441,9 +492,59 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>+		MenuManagerRenderer renderer = getRenderer(appContext, menuBar);
>+
>+		MenuManager fileManager = renderer.getManager(fileMenu);
>+		assertNotNull("No file menu?", fileManager);
>+
>+		assertEquals(4, fileManager.getSize());
>+
>+		IContributionItem mmcItem = fileManager.getItems()[3];
>+		assertEquals("mmc.item1", mmcItem.getId());
>+		assertEquals("before the first show, we have no context to evaluate",
>+				true, mmcItem.isVisible());
>+
>+		MenuManager manager = renderer.getManager(menuBar);
>+		manager.updateAll(true);
>+		Menu fileWidget = fileManager.getMenu();
>+		assertNotNull(fileWidget);
>+
>+		Event show = new Event();
>+		show.widget = fileWidget;
>+		show.type = SWT.Show;
>+
>+		Event hide = new Event();
>+		hide.widget = fileWidget;
>+		hide.type = SWT.Hide;
>+
>+		fileWidget.notifyListeners(SWT.Show, show);
>+
>+		assertEquals("after the first show, it should not be visible", false,
>+				mmcItem.isVisible());
>+
>+		fileWidget.notifyListeners(SWT.Hide, hide);
>+
>+		appContext.set("mmc1", Boolean.TRUE);
>+
>+		assertEquals("Change should not show up until next show", false,
>+				mmcItem.isVisible());
>+
>+		fileWidget.notifyListeners(SWT.Show, show);
>+
>+		assertEquals(true, mmcItem.isVisible());
>+
>+		fileWidget.notifyListeners(SWT.Hide, hide);
>+
>+		appContext.remove("mmc1");
>+
>+		fileWidget.notifyListeners(SWT.Show, show);
>+
>+		assertEquals(false, mmcItem.isVisible());
>+
>+		fileWidget.notifyListeners(SWT.Hide, hide);
> 	}
> 
>-	public void XXXtestMenuBarVisibility() throws Exception {
>+	public void testMenuBarVisibility() throws Exception {
> 		MWindow window = BasicFactoryImpl.eINSTANCE.createWindow();
> 		MMenu menuBar = MenuFactoryImpl.eINSTANCE.createMenu();
> 		menuBar.setElementId("org.eclipse.ui.main.menu");
>@@ -478,6 +579,36 @@
> 		wb = new E4Workbench(window, appContext);
> 		wb.createAndRunUI(window);
> 
>+		MenuManagerRenderer renderer = getRenderer(appContext, menuBar);
>+		MenuManager manager = renderer.getManager(menuBar);
>+		manager.updateAll(true);
>+
>+		assertEquals(2, manager.getSize());
>+
>+		MenuManager vanishManager = (MenuManager) manager.getItems()[1];
>+		assertEquals("vanish", vanishManager.getId());
>+
>+		assertFalse(vanishManager.isVisible());
>+		assertNull(vanishManager.getMenu());
>+
>+		appContext.set("mmc1", Boolean.TRUE);
>+
>+		assertTrue(vanishManager.isVisible());
>+		assertNotNull(vanishManager.getMenu());
>+
>+		appContext.remove("mmc1");
>+
>+		assertFalse(vanishManager.isVisible());
>+		Menu vanishMenu = vanishManager.getMenu();
>+		if (vanishMenu != null) {
>+			assertTrue(vanishMenu.isDisposed());
>+		}
>+
>+		appContext.set("mmc1", Boolean.TRUE);
>+
>+		assertTrue(vanishManager.isVisible());
>+		assertNotNull(vanishManager.getMenu());
>+		assertFalse(vanishManager.getMenu().isDisposed());
> 	}
> 
> 	private MMenuContribution createContribution(boolean withVisibleWhen) {
>@@ -537,11 +668,12 @@
> 		application.getMenuContributions().add(mmc);
> 	}
> 
>-	MenuRenderer getRenderer(IEclipseContext context, MUIElement element) {
>+	private MenuManagerRenderer getRenderer(IEclipseContext context,
>+			MUIElement element) {
> 		IRendererFactory rendererFactory = context.get(IRendererFactory.class);
> 		AbstractPartRenderer renderer = rendererFactory.getRenderer(element,
> 				null);
>-		assertEquals(MenuRenderer.class, renderer.getClass());
>-		return (MenuRenderer) renderer;
>+		assertEquals(MenuManagerRenderer.class, renderer.getClass());
>+		return (MenuManagerRenderer) renderer;
> 	}
> }
>Index: src/org/eclipse/e4/ui/tests/workbench/MWindowTest.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/tests/org.eclipse.e4.ui.tests/src/org/eclipse/e4/ui/tests/workbench/MWindowTest.java,v
>retrieving revision 1.45
>diff -u -r1.45 MWindowTest.java
>--- src/org/eclipse/e4/ui/tests/workbench/MWindowTest.java	14 Oct 2010 18:21:17 -0000	1.45
>+++ src/org/eclipse/e4/ui/tests/workbench/MWindowTest.java	19 Oct 2010 15:22:31 -0000
>@@ -30,6 +30,7 @@
> import org.eclipse.e4.ui.model.application.ui.menu.impl.MenuFactoryImpl;
> import org.eclipse.e4.ui.services.IServiceConstants;
> import org.eclipse.e4.ui.widgets.CTabFolder;
>+import org.eclipse.jface.action.MenuManager;
> import org.eclipse.swt.SWT;
> import org.eclipse.swt.graphics.Rectangle;
> import org.eclipse.swt.widgets.Composite;
>@@ -230,6 +231,8 @@
> 
> 		wb = new E4Workbench(application, appContext);
> 		wb.createAndRunUI(window);
>+		((MenuManager) ((Widget) window.getMainMenu().getWidget()).getData())
>+				.updateAll(true);
> 
> 		Widget topWidget = (Widget) window.getWidget();
> 		assertNotNull(topWidget);
>#P org.eclipse.e4.ui.workbench
>Index: src/org/eclipse/e4/ui/internal/workbench/ContributionsAnalyzer.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench/src/org/eclipse/e4/ui/internal/workbench/ContributionsAnalyzer.java,v
>retrieving revision 1.18
>diff -u -r1.18 ContributionsAnalyzer.java
>--- src/org/eclipse/e4/ui/internal/workbench/ContributionsAnalyzer.java	14 Oct 2010 18:21:18 -0000	1.18
>+++ src/org/eclipse/e4/ui/internal/workbench/ContributionsAnalyzer.java	19 Oct 2010 15:22:32 -0000
>@@ -176,7 +176,7 @@
> 		return isVisible((MCoreExpression) contribution.getVisibleWhen(), eContext);
> 	}
> 
>-	static boolean isVisible(MCoreExpression exp, ExpressionContext eContext) {
>+	public static boolean isVisible(MCoreExpression exp, ExpressionContext eContext) {
> 		Expression ref = null;
> 		if (exp.getCoreExpression() instanceof Expression) {
> 			ref = (Expression) exp.getCoreExpression();
>#P org.eclipse.e4.ui.workbench.renderers.swt
>Index: src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerRenderer.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerRenderer.java,v
>retrieving revision 1.1
>diff -u -r1.1 MenuManagerRenderer.java
>--- src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerRenderer.java	14 Oct 2010 18:21:18 -0000	1.1
>+++ src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerRenderer.java	19 Oct 2010 15:22:33 -0000
>@@ -25,9 +25,7 @@
> import org.eclipse.e4.core.services.events.IEventBroker;
> import org.eclipse.e4.core.services.log.Logger;
> import org.eclipse.e4.ui.internal.workbench.ContributionsAnalyzer;
>-import org.eclipse.e4.ui.internal.workbench.swt.AbstractPartRenderer;
> import org.eclipse.e4.ui.model.application.MApplication;
>-import org.eclipse.e4.ui.model.application.ui.MContext;
> import org.eclipse.e4.ui.model.application.ui.MElementContainer;
> import org.eclipse.e4.ui.model.application.ui.MUIElement;
> import org.eclipse.e4.ui.model.application.ui.MUILabel;
>@@ -48,17 +46,18 @@
> import org.eclipse.emf.ecore.EObject;
> import org.eclipse.emf.ecore.util.EcoreUtil;
> import org.eclipse.jface.action.AbstractGroupMarker;
>+import org.eclipse.jface.action.ContributionItem;
> import org.eclipse.jface.action.GroupMarker;
> import org.eclipse.jface.action.IContributionItem;
>-import org.eclipse.jface.action.IMenuListener;
>-import org.eclipse.jface.action.IMenuManager;
> import org.eclipse.jface.action.MenuManager;
> import org.eclipse.jface.action.Separator;
> import org.eclipse.jface.resource.ImageDescriptor;
>+import org.eclipse.swt.SWT;
> import org.eclipse.swt.events.DisposeEvent;
> import org.eclipse.swt.events.DisposeListener;
> import org.eclipse.swt.widgets.Control;
> import org.eclipse.swt.widgets.Decorations;
>+import org.eclipse.swt.widgets.Display;
> import org.eclipse.swt.widgets.Menu;
> import org.osgi.service.event.Event;
> import org.osgi.service.event.EventHandler;
>@@ -145,13 +144,20 @@
> 						return;
> 					}
> 					manager.setVisible(menuModel.isVisible());
>+					if (manager.getParent() != null) {
>+						manager.getParent().markDirty();
>+					}
> 				} else if (element instanceof MMenuElement) {
> 					MMenuElement itemModel = (MMenuElement) element;
>-					IContributionItem ici = getContribution(itemModel);
>-					if (ici == null) {
>+					Object ici = getContribution(itemModel);
>+					if (!(ici instanceof ContributionItem)) {
> 						return;
> 					}
>-					ici.setVisible(itemModel.isVisible());
>+					ContributionItem item = (ContributionItem) ici;
>+					item.setVisible(itemModel.isVisible());
>+					if (item.getParent() != null) {
>+						item.getParent().markDirty();
>+					}
> 				}
> 			}
> 		}
>@@ -187,43 +193,7 @@
> 		}
> 	};
> 
>-	private IMenuListener visibilityCalculationListener = new IMenuListener() {
>-		public void menuAboutToShow(IMenuManager manager) {
>-			MenuManager menuManager = (MenuManager) manager;
>-			if (menuManager.getMenu() == null) {
>-				return;
>-			}
>-			MMenu menuModel = getMenuModel(menuManager);
>-			if (menuModel == null) {
>-				Menu menu = menuManager.getMenu();
>-				Object obj = menu.getData(AbstractPartRenderer.OWNING_ME);
>-				if (obj == null && menu.getParentItem() != null) {
>-					obj = menu.getParentItem().getData(
>-							AbstractPartRenderer.OWNING_ME);
>-				}
>-				if (!(obj instanceof MMenu)) {
>-					return;
>-				}
>-				menuModel = (MMenu) obj;
>-			} else if (menuModel.getWidget() == null) {
>-				bindWidget(menuModel, menuManager.getMenu());
>-			}
>-			final IEclipseContext evalContext;
>-			if (menuModel instanceof MContext) {
>-				evalContext = ((MContext) menuModel).getContext();
>-			} else {
>-				evalContext = modelService.getContainingContext(menuModel);
>-			}
>-			HashSet<ContributionRecord> records = new HashSet<ContributionRecord>();
>-			for (MMenuElement element : menuModel.getChildren()) {
>-				ContributionRecord record = modelContributionToRecord
>-						.get(element);
>-				if (record != null && records.add(record)) {
>-					record.updateVisibility(evalContext);
>-				}
>-			}
>-		}
>-	};
>+	private MenuManagerRendererFilter rendererFilter;
> 
> 	@PostConstruct
> 	public void init() {
>@@ -237,6 +207,14 @@
> 		eventBroker.subscribe(UIEvents.buildTopic(UIEvents.UIElement.TOPIC),
> 				toBeRenderedUpdater);
> 
>+		Display display = context.get(Display.class);
>+		rendererFilter = ContextInjectionFactory.make(
>+				MenuManagerRendererFilter.class, context);
>+		display.addFilter(SWT.Show, rendererFilter);
>+		display.addFilter(SWT.Hide, rendererFilter);
>+		display.addFilter(SWT.Dispose, rendererFilter);
>+		context.set(MenuManagerRendererFilter.class, rendererFilter);
>+
> 	}
> 
> 	@PreDestroy
>@@ -245,6 +223,16 @@
> 		eventBroker.unsubscribe(selectionUpdater);
> 		eventBroker.unsubscribe(enabledUpdater);
> 		eventBroker.unsubscribe(toBeRenderedUpdater);
>+
>+		Display display = context.get(Display.class);
>+		if (display != null && !display.isDisposed()) {
>+			display.removeFilter(SWT.Show, rendererFilter);
>+			display.removeFilter(SWT.Hide, rendererFilter);
>+			display.removeFilter(SWT.Dispose, rendererFilter);
>+			ContextInjectionFactory.uninject(rendererFilter, context);
>+			context.remove(MenuManagerRendererFilter.class);
>+			rendererFilter = null;
>+		}
> 	}
> 
> 	/*
>@@ -686,18 +674,18 @@
> 	public void linkModelToManager(MMenu model, MenuManager manager) {
> 		modelToManager.put(model, manager);
> 		managerToModel.put(manager, model);
>-		manager.addMenuListener(visibilityCalculationListener);
> 	}
> 
> 	public void clearModelToManager(MMenu model, MenuManager manager) {
> 		modelToManager.remove(model);
> 		managerToModel.remove(manager);
>-		if (manager != null) {
>-			manager.removeMenuListener(visibilityCalculationListener);
>-		}
> 	}
> 
> 	public IContributionItem getContribution(MMenuElement model) {
> 		return modelToContribution.get(model);
> 	}
>+
>+	public ContributionRecord getContributionRecord(MMenuElement element) {
>+		return modelContributionToRecord.get(element);
>+	}
> }
>Index: src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerRendererFilter.java
>===================================================================
>RCS file: src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerRendererFilter.java
>diff -N src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerRendererFilter.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerRendererFilter.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,434 @@
>+/*******************************************************************************
>+ * Copyright (c) 2010 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
>+ * http://www.eclipse.org/legal/epl-v10.html
>+ *
>+ * Contributors:
>+ *     IBM Corporation - initial API and implementation
>+ *******************************************************************************/
>+package org.eclipse.e4.ui.workbench.renderers.swt;
>+
>+import java.lang.reflect.InvocationTargetException;
>+import java.lang.reflect.Method;
>+import java.util.ArrayList;
>+import java.util.HashMap;
>+import java.util.HashSet;
>+import java.util.List;
>+import javax.inject.Inject;
>+import org.eclipse.core.commands.ParameterizedCommand;
>+import org.eclipse.core.runtime.ISafeRunnable;
>+import org.eclipse.core.runtime.SafeRunner;
>+import org.eclipse.e4.core.commands.EHandlerService;
>+import org.eclipse.e4.core.contexts.IEclipseContext;
>+import org.eclipse.e4.core.services.log.Logger;
>+import org.eclipse.e4.ui.internal.workbench.ContributionsAnalyzer;
>+import org.eclipse.e4.ui.internal.workbench.swt.AbstractPartRenderer;
>+import org.eclipse.e4.ui.internal.workbench.swt.Policy;
>+import org.eclipse.e4.ui.internal.workbench.swt.WorkbenchSWTActivator;
>+import org.eclipse.e4.ui.model.application.MApplication;
>+import org.eclipse.e4.ui.model.application.ui.MContext;
>+import org.eclipse.e4.ui.model.application.ui.MCoreExpression;
>+import org.eclipse.e4.ui.model.application.ui.menu.MHandledMenuItem;
>+import org.eclipse.e4.ui.model.application.ui.menu.MMenu;
>+import org.eclipse.e4.ui.model.application.ui.menu.MMenuContribution;
>+import org.eclipse.e4.ui.model.application.ui.menu.MMenuElement;
>+import org.eclipse.e4.ui.model.application.ui.menu.MPopupMenu;
>+import org.eclipse.e4.ui.model.application.ui.menu.MRenderedMenu;
>+import org.eclipse.e4.ui.workbench.modeling.EModelService;
>+import org.eclipse.e4.ui.workbench.modeling.ExpressionContext;
>+import org.eclipse.e4.ui.workbench.renderers.swt.MenuManagerRenderer.ContributionRecord;
>+import org.eclipse.e4.ui.workbench.swt.factories.IRendererFactory;
>+import org.eclipse.jface.action.MenuManager;
>+import org.eclipse.swt.SWT;
>+import org.eclipse.swt.widgets.Event;
>+import org.eclipse.swt.widgets.Listener;
>+import org.eclipse.swt.widgets.Menu;
>+import org.eclipse.swt.widgets.Widget;
>+
>+public class MenuManagerRendererFilter implements Listener {
>+	public static final String NUL_MENU_ITEM = "(None Applicable)"; //$NON-NLS-1$
>+
>+	private static final String TMP_ORIGINAL_CONTEXT = "MenuServiceFilter.original.context"; //$NON-NLS-1$
>+
>+	private static void trace(String msg, Widget menu, MMenu menuModel) {
>+		WorkbenchSWTActivator.trace(Policy.MENUS, msg + ": " + menu + ": " //$NON-NLS-1$ //$NON-NLS-2$
>+				+ menuModel, null);
>+	}
>+
>+	private static Method aboutToShow;
>+
>+	private static Method aboutToHide;
>+
>+	public static Method getAboutToShow() {
>+		if (aboutToShow == null) {
>+			try {
>+				aboutToShow = MenuManager.class
>+						.getDeclaredMethod("handleAboutToShow"); //$NON-NLS-1$
>+				aboutToShow.setAccessible(true);
>+			} catch (SecurityException e) {
>+				// TODO Auto-generated catch block
>+				e.printStackTrace();
>+			} catch (NoSuchMethodException e) {
>+				// TODO Auto-generated catch block
>+				e.printStackTrace();
>+			}
>+		}
>+		return aboutToShow;
>+	}
>+
>+	public static Method getAboutToHide() {
>+		if (aboutToHide == null) {
>+			try {
>+				aboutToHide = MenuManager.class
>+						.getDeclaredMethod("handleAboutToHide"); //$NON-NLS-1$
>+				aboutToHide.setAccessible(true);
>+			} catch (SecurityException e) {
>+				// TODO Auto-generated catch block
>+				e.printStackTrace();
>+			} catch (NoSuchMethodException e) {
>+				// TODO Auto-generated catch block
>+				e.printStackTrace();
>+			}
>+		}
>+		return aboutToHide;
>+	}
>+
>+	@Inject
>+	private MApplication application;
>+
>+	@Inject
>+	private Logger logger;
>+
>+	@Inject
>+	private EModelService modelService;
>+
>+	@Inject
>+	private IRendererFactory rendererFactory;
>+
>+	private HashMap<Menu, Runnable> pendingCleanup = new HashMap<Menu, Runnable>();
>+
>+	private class SafeWrapper implements ISafeRunnable {
>+		Event event;
>+
>+		public void handleException(Throwable e) {
>+			if (e instanceof Error) {
>+				// errors are deadly, we shouldn't ignore these
>+				throw (Error) e;
>+			}
>+			// log exceptions otherwise
>+			if (logger != null) {
>+				logger.error(e);
>+			}
>+		}
>+
>+		public void run() throws Exception {
>+			safeHandleEvent(event);
>+		}
>+	}
>+
>+	private SafeWrapper safeWrapper = new SafeWrapper();
>+
>+	public void handleEvent(final Event event) {
>+		// wrap the handling in a SafeRunner so that exceptions do not prevent
>+		// the menu from being shown
>+		safeWrapper.event = event;
>+		SafeRunner.run(safeWrapper);
>+	}
>+
>+	private void safeHandleEvent(Event event) {
>+		if (!(event.widget instanceof Menu)) {
>+			return;
>+		}
>+		final Menu menu = (Menu) event.widget;
>+		if (event.type == SWT.Dispose) {
>+			trace("handleMenu.Dispose", menu, null); //$NON-NLS-1$
>+			cleanUp(menu);
>+		}
>+		Object obj = menu.getData(AbstractPartRenderer.OWNING_ME);
>+		if (obj == null && menu.getParentItem() != null) {
>+			obj = menu.getParentItem().getData(AbstractPartRenderer.OWNING_ME);
>+		}
>+		if (obj instanceof MPopupMenu) {
>+			handleContextMenu(event, menu, (MPopupMenu) obj);
>+		} else if (obj instanceof MMenu) {
>+			handleMenu(event, menu, (MMenu) obj);
>+		} else {
>+			trace("Incorrect menu model to work with: " + obj, menu, null); //$NON-NLS-1$
>+		}
>+	}
>+
>+	private void handleMenu(final Event event, final Menu menu,
>+			final MMenu menuModel) {
>+		if ((menu.getStyle() & SWT.BAR) != 0) {
>+			// don't process the menu bar, it's not fair :-)
>+			return;
>+		}
>+		switch (event.type) {
>+		case SWT.Show:
>+			trace("handleMenu.Show", menu, menuModel); //$NON-NLS-1$
>+			cleanUp(menu);
>+			showMenu(event, menu, menuModel);
>+			break;
>+		case SWT.Hide:
>+			trace("handleMenu.Hide", menu, menuModel); //$NON-NLS-1$
>+			// TODO we'll clean up on show
>+			break;
>+		}
>+	}
>+
>+	public void showMenu(final Event event, final Menu menu,
>+			final MMenu menuModel) {
>+		AbstractPartRenderer obj = rendererFactory.getRenderer(menuModel,
>+				menu.getParent());
>+		if (!(obj instanceof MenuManagerRenderer)) {
>+			trace("Not the correct renderer: " + obj, menu, menuModel); //$NON-NLS-1$
>+			return;
>+		}
>+		MenuManagerRenderer renderer = (MenuManagerRenderer) obj;
>+		MenuManager menuManager = renderer.getManager(menuModel);
>+		if (menuModel.getWidget() == null) {
>+			renderer.bindWidget(menuModel, menuManager.getMenu());
>+		}
>+
>+		Method handleAboutToShow = getAboutToShow();
>+		try {
>+			handleAboutToShow.invoke(menuManager);
>+		} catch (IllegalArgumentException e) {
>+			// TODO Auto-generated catch block
>+			e.printStackTrace();
>+		} catch (IllegalAccessException e) {
>+			// TODO Auto-generated catch block
>+			e.printStackTrace();
>+		} catch (InvocationTargetException e) {
>+			// TODO Auto-generated catch block
>+			e.printStackTrace();
>+		}
>+
>+		//
>+		// TODO merge MRenderedMenuItems back into the model
>+		// this will make the 3.x world visible to the model
>+		// without actually modeling it fully
>+		//
>+
>+		final IEclipseContext evalContext;
>+		if (menuModel instanceof MContext) {
>+			evalContext = ((MContext) menuModel).getContext();
>+		} else {
>+			evalContext = modelService.getContainingContext(menuModel);
>+		}
>+		updateElementVisibility(menuModel, renderer, menuManager, evalContext,
>+				true);
>+
>+		// last thing to do, kill the event and update the menu manager
>+		event.type = SWT.None;
>+		event.doit = false;
>+		menuManager.update(false);
>+	}
>+
>+	/**
>+	 * @param menuModel
>+	 * @param renderer
>+	 * @param menuManager
>+	 * @param evalContext
>+	 */
>+	private void updateElementVisibility(final MMenu menuModel,
>+			MenuManagerRenderer renderer, MenuManager menuManager,
>+			final IEclipseContext evalContext, boolean recurse) {
>+		final ExpressionContext exprContext = new ExpressionContext(evalContext);
>+		HashSet<ContributionRecord> records = new HashSet<ContributionRecord>();
>+		for (MMenuElement element : menuModel.getChildren()) {
>+			ContributionRecord record = renderer.getContributionRecord(element);
>+			if (record != null) {
>+				if (records.add(record)) {
>+					record.updateVisibility(evalContext);
>+				}
>+			} else {
>+				updateVisibility(menuManager, element, exprContext);
>+			}
>+			if (recurse && element.isVisible() && element instanceof MMenu) {
>+				MMenu childMenu = (MMenu) element;
>+				MenuManager childManager = renderer.getManager(childMenu);
>+				if (childManager != null) {
>+					updateElementVisibility(childMenu, renderer, childManager,
>+							evalContext, false);
>+				}
>+			}
>+		}
>+	}
>+
>+	/**
>+	 * @param menuManager
>+	 * @param element
>+	 * @param evalContext
>+	 */
>+	private void updateVisibility(MenuManager menuManager,
>+			MMenuElement element, ExpressionContext evalContext) {
>+		if (!(element.getVisibleWhen() instanceof MCoreExpression)) {
>+			return;
>+		}
>+		boolean val = ContributionsAnalyzer.isVisible(
>+				(MCoreExpression) element.getVisibleWhen(), evalContext);
>+		if (val != element.isVisible()) {
>+			element.setVisible(val);
>+			menuManager.markDirty();
>+		}
>+	}
>+
>+	private void handleContextMenu(final Event event, final Menu menu,
>+			final MPopupMenu menuModel) {
>+		switch (event.type) {
>+		case SWT.Show:
>+			trace("handleContextMenu.Show", menu, menuModel); //$NON-NLS-1$
>+			cleanUp(menu);
>+			showPopup(event, menu, menuModel);
>+			break;
>+		case SWT.Hide:
>+			trace("handleContextMenu.Hide", menu, menuModel); //$NON-NLS-1$
>+			hidePopup(event, menu, menuModel);
>+			break;
>+		}
>+	}
>+
>+	public void hidePopup(Event event, Menu menu, MPopupMenu menuModel) {
>+		final IEclipseContext popupContext = menuModel.getContext();
>+		final IEclipseContext originalChild = (IEclipseContext) popupContext
>+				.get(TMP_ORIGINAL_CONTEXT);
>+		popupContext.remove(TMP_ORIGINAL_CONTEXT);
>+		if (!menu.isDisposed()) {
>+			menu.getDisplay().asyncExec(new Runnable() {
>+				public void run() {
>+					if (originalChild == null) {
>+						popupContext.deactivate();
>+					} else {
>+						originalChild.activate();
>+					}
>+				}
>+			});
>+		}
>+	}
>+
>+	public void showPopup(final Event event, final Menu menu,
>+			final MPopupMenu menuModel) {
>+		// System.err.println("showPopup: " + menuModel + "\n\t" + menu);
>+		// we need some context foolery here
>+		final IEclipseContext popupContext = menuModel.getContext();
>+		final IEclipseContext parentContext = popupContext.getParent();
>+		final IEclipseContext originalChild = parentContext.getActiveChild();
>+		popupContext.activate();
>+		popupContext.set(TMP_ORIGINAL_CONTEXT, originalChild);
>+
>+		final ArrayList<MMenuContribution> toContribute = new ArrayList<MMenuContribution>();
>+		final ArrayList<MMenuElement> menuContributionsToRemove = new ArrayList<MMenuElement>();
>+		ExpressionContext eContext = new ExpressionContext(popupContext);
>+		ContributionsAnalyzer.gatherMenuContributions(menuModel,
>+				application.getMenuContributions(), menuModel.getElementId(),
>+				toContribute, eContext, true);
>+
>+		for (String tag : menuModel.getTags()) {
>+			if (tag.startsWith("popup:") && tag.length() > 6) { //$NON-NLS-1$
>+				ContributionsAnalyzer.gatherMenuContributions(menuModel,
>+						application.getMenuContributions(), tag.substring(6),
>+						toContribute, eContext, false);
>+			}
>+		}
>+		ContributionsAnalyzer.addMenuContributions(menuModel, toContribute,
>+				menuContributionsToRemove);
>+
>+		// create a cleanup routine for the Hide or next Show
>+		pendingCleanup.put(menu, new Runnable() {
>+			public void run() {
>+				if (!menu.isDisposed()) {
>+					unrender(menuContributionsToRemove);
>+				}
>+				removeMenuContributions(menuModel, menuContributionsToRemove);
>+			}
>+		});
>+		render(menu, menuModel);
>+	}
>+
>+	private void render(final Menu menu, final MMenu menuModel) {
>+		trace("render", menu, menuModel); //$NON-NLS-1$
>+
>+	}
>+
>+	void setEnabled(MHandledMenuItem item) {
>+		if (!item.isToBeRendered() || !item.isVisible()
>+				|| item.getWidget() == null) {
>+			return;
>+		}
>+		ParameterizedCommand cmd = item.getWbCommand();
>+		if (cmd == null) {
>+			return;
>+		}
>+		final IEclipseContext lclContext = modelService
>+				.getContainingContext(item);
>+		EHandlerService service = lclContext.get(EHandlerService.class);
>+		item.setEnabled(service.canExecute(cmd));
>+	}
>+
>+	private void unrender(final List<MMenuElement> menuModel) {
>+		trace("unrender", null, null); //$NON-NLS-1$
>+	}
>+
>+	private void removeMenuContributions(final MMenu menuModel,
>+			final ArrayList<MMenuElement> menuContributionsToRemove) {
>+		for (MMenuElement item : menuContributionsToRemove) {
>+			trace("removeMenuContributions " + item, //$NON-NLS-1$
>+					(Widget) menuModel.getWidget(), menuModel);
>+			menuModel.getChildren().remove(item);
>+		}
>+	}
>+
>+	public void showRenderedMenu(final Event event, final Menu menu,
>+			final MRenderedMenu menuModel) {
>+		if (!(menuModel.getContributionManager() instanceof MenuManager)) {
>+			return;
>+		}
>+
>+		MenuManager manager = (MenuManager) menuModel.getContributionManager();
>+		Method handleAboutToShow = getAboutToShow();
>+		try {
>+			handleAboutToShow.invoke(manager);
>+		} catch (IllegalArgumentException e) {
>+			// TODO Auto-generated catch block
>+			e.printStackTrace();
>+		} catch (IllegalAccessException e) {
>+			// TODO Auto-generated catch block
>+			e.printStackTrace();
>+		} catch (InvocationTargetException e) {
>+			// TODO Auto-generated catch block
>+			e.printStackTrace();
>+		}
>+
>+		if (menuModel.getChildren().size() == 1
>+				&& menuModel.getChildren().get(0) instanceof MPopupMenu) {
>+			showPopup(event, menu, (MPopupMenu) menuModel.getChildren().get(0));
>+		} else {
>+			showMenu(event, menu, menuModel);
>+		}
>+		event.type = SWT.None;
>+		event.doit = false;
>+	}
>+
>+	public void cleanUp(final Menu menu) {
>+		trace("cleanUp", menu, null); //$NON-NLS-1$
>+		if (pendingCleanup.isEmpty()) {
>+			return;
>+		}
>+		Runnable cleanUp = pendingCleanup.remove(menu);
>+		if (cleanUp != null) {
>+			trace("cleanUp.run()", menu, null); //$NON-NLS-1$
>+			cleanUp.run();
>+		}
>+	}
>+
>+	public void dispose() {
>+		Menu[] keys = pendingCleanup.keySet().toArray(
>+				new Menu[pendingCleanup.size()]);
>+		for (Menu menu : keys) {
>+			cleanUp(menu);
>+		}
>+	}
>+}
>Index: src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerServiceFilter.java
>===================================================================
>RCS file: src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerServiceFilter.java
>diff -N src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerServiceFilter.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerServiceFilter.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,182 @@
>+/*******************************************************************************
>+ * Copyright (c) 2010 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
>+ * http://www.eclipse.org/legal/epl-v10.html
>+ *
>+ * Contributors:
>+ *     IBM Corporation - initial API and implementation
>+ *******************************************************************************/
>+package org.eclipse.e4.ui.workbench.renderers.swt;
>+
>+import java.lang.reflect.Method;
>+import javax.inject.Inject;
>+import org.eclipse.core.runtime.ISafeRunnable;
>+import org.eclipse.core.runtime.SafeRunner;
>+import org.eclipse.e4.core.contexts.IEclipseContext;
>+import org.eclipse.e4.core.services.log.Logger;
>+import org.eclipse.e4.ui.internal.workbench.swt.AbstractPartRenderer;
>+import org.eclipse.e4.ui.internal.workbench.swt.Policy;
>+import org.eclipse.e4.ui.internal.workbench.swt.WorkbenchSWTActivator;
>+import org.eclipse.e4.ui.model.application.ui.menu.MMenu;
>+import org.eclipse.e4.ui.model.application.ui.menu.MPopupMenu;
>+import org.eclipse.e4.ui.workbench.modeling.EModelService;
>+import org.eclipse.jface.action.MenuManager;
>+import org.eclipse.swt.SWT;
>+import org.eclipse.swt.widgets.Event;
>+import org.eclipse.swt.widgets.Listener;
>+import org.eclipse.swt.widgets.Menu;
>+import org.eclipse.swt.widgets.Widget;
>+
>+public class MenuManagerServiceFilter implements Listener {
>+	public static final String NUL_MENU_ITEM = "(None Applicable)"; //$NON-NLS-1$
>+
>+	private static final String TMP_ORIGINAL_CONTEXT = "MenuServiceFilter.original.context"; //$NON-NLS-1$
>+
>+	private static void trace(String msg, Widget menu, MMenu menuModel) {
>+		WorkbenchSWTActivator.trace(Policy.MENUS, msg + ": " + menu + ": " //$NON-NLS-1$ //$NON-NLS-2$
>+				+ menuModel, null);
>+	}
>+
>+	private static Method aboutToShow;
>+
>+	public static Method getAboutToShow() {
>+		if (aboutToShow == null) {
>+			try {
>+				aboutToShow = MenuManager.class
>+						.getDeclaredMethod("handleAboutToShow"); //$NON-NLS-1$
>+				aboutToShow.setAccessible(true);
>+			} catch (SecurityException e) {
>+				// TODO Auto-generated catch block
>+				e.printStackTrace();
>+			} catch (NoSuchMethodException e) {
>+				// TODO Auto-generated catch block
>+				e.printStackTrace();
>+			}
>+		}
>+		return aboutToShow;
>+	}
>+
>+	@Inject
>+	private Logger logger;
>+
>+	@Inject
>+	EModelService modelService;
>+
>+	public void handleEvent(final Event event) {
>+		// wrap the handling in a SafeRunner so that exceptions do not prevent
>+		// the menu from being shown
>+		SafeRunner.run(new ISafeRunnable() {
>+			public void handleException(Throwable e) {
>+				if (e instanceof Error) {
>+					// errors are deadly, we shouldn't ignore these
>+					throw (Error) e;
>+				} else {
>+					// log exceptions otherwise
>+					if (logger != null) {
>+						logger.error(e);
>+					}
>+				}
>+			}
>+
>+			public void run() throws Exception {
>+				safeHandleEvent(event);
>+			}
>+		});
>+	}
>+
>+	private void safeHandleEvent(Event event) {
>+		if (!(event.widget instanceof Menu)) {
>+			return;
>+		}
>+		final Menu menu = (Menu) event.widget;
>+		if (event.type == SWT.Dispose) {
>+			trace("handleMenu.Dispose", menu, null); //$NON-NLS-1$
>+			cleanUp(menu);
>+		}
>+		Object obj = menu.getData(AbstractPartRenderer.OWNING_ME);
>+		if (obj == null && menu.getParentItem() != null) {
>+			obj = menu.getParentItem().getData(AbstractPartRenderer.OWNING_ME);
>+		}
>+		if (obj instanceof MPopupMenu) {
>+			handleContextMenu(event, menu, (MPopupMenu) obj);
>+		} else if (obj instanceof MMenu) {
>+			handleMenu(event, menu, (MMenu) obj);
>+		}
>+	}
>+
>+	private void handleMenu(final Event event, final Menu menu,
>+			final MMenu menuModel) {
>+		if ((menu.getStyle() & SWT.BAR) != 0) {
>+			// don't process the menu bar, it's not fair :-)
>+			return;
>+		}
>+		switch (event.type) {
>+		case SWT.Show:
>+			cleanUp(menu);
>+			showMenu(event, menu, menuModel);
>+			break;
>+		case SWT.Hide:
>+			// TODO we'll clean up on show
>+			break;
>+		}
>+	}
>+
>+	public void showMenu(final Event event, final Menu menu,
>+			final MMenu menuModel) {
>+		// System.err.println("showMenu: " + menuModel + "\n\t" + menu);
>+	}
>+
>+	private void handleContextMenu(final Event event, final Menu menu,
>+			final MPopupMenu menuModel) {
>+		switch (event.type) {
>+		case SWT.Show:
>+			cleanUp(menu);
>+			showPopup(event, menu, menuModel);
>+			break;
>+		case SWT.Hide:
>+			hidePopup(event, menu, menuModel);
>+			break;
>+		}
>+	}
>+
>+	public void hidePopup(Event event, Menu menu, MPopupMenu menuModel) {
>+		// System.err.println("hidePopup: " + menuModel + "\n\t" + menu);
>+		final IEclipseContext popupContext = menuModel.getContext();
>+		final IEclipseContext originalChild = (IEclipseContext) popupContext
>+				.get(TMP_ORIGINAL_CONTEXT);
>+		popupContext.remove(TMP_ORIGINAL_CONTEXT);
>+		if (!menu.isDisposed()) {
>+			menu.getDisplay().asyncExec(new Runnable() {
>+				public void run() {
>+					if (originalChild == null) {
>+						popupContext.deactivate();
>+					} else {
>+						originalChild.activate();
>+					}
>+				}
>+			});
>+		}
>+	}
>+
>+	public void showPopup(final Event event, final Menu menu,
>+			final MPopupMenu menuModel) {
>+		// System.err.println("showPopup: " + menuModel + "\n\t" + menu);
>+		// we need some context foolery here
>+		final IEclipseContext popupContext = menuModel.getContext();
>+		final IEclipseContext parentContext = popupContext.getParent();
>+		final IEclipseContext originalChild = parentContext.getActiveChild();
>+		popupContext.activate();
>+		popupContext.set(TMP_ORIGINAL_CONTEXT, originalChild);
>+
>+	}
>+
>+	public void cleanUp(final Menu menu) {
>+		// System.err.println("cleanUp: " + menu);
>+	}
>+
>+	public void dispose() {
>+		// System.err.println("dispose");
>+	}
>+}
>Index: src/org/eclipse/e4/ui/workbench/renderers/swt/MenuRenderer.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/MenuRenderer.java,v
>retrieving revision 1.10
>diff -u -r1.10 MenuRenderer.java
>--- src/org/eclipse/e4/ui/workbench/renderers/swt/MenuRenderer.java	14 Oct 2010 18:21:18 -0000	1.10
>+++ src/org/eclipse/e4/ui/workbench/renderers/swt/MenuRenderer.java	19 Oct 2010 15:22:33 -0000
>@@ -25,7 +25,6 @@
> import org.eclipse.e4.ui.model.application.ui.menu.MMenuElement;
> import org.eclipse.e4.ui.model.application.ui.menu.MMenuSeparator;
> import org.eclipse.e4.ui.workbench.modeling.ExpressionContext;
>-import org.eclipse.e4.ui.workbench.swt.modeling.MenuServiceFilter;
> import org.eclipse.emf.ecore.EObject;
> import org.eclipse.swt.SWT;
> import org.eclipse.swt.events.DisposeEvent;
>@@ -102,7 +101,7 @@
> 				}
> 				Menu menu = mi.getMenu();
> 				MenuItem menuItem = new MenuItem(menu, SWT.PUSH);
>-				menuItem.setText(MenuServiceFilter.NUL_MENU_ITEM);
>+				menuItem.setText(MenuManagerRendererFilter.NUL_MENU_ITEM);
> 				menuItem.setEnabled(false);
> 			}
> 		}
>Index: src/org/eclipse/e4/ui/workbench/renderers/swt/SeparatorRenderer.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/SeparatorRenderer.java,v
>retrieving revision 1.10
>diff -u -r1.10 SeparatorRenderer.java
>--- src/org/eclipse/e4/ui/workbench/renderers/swt/SeparatorRenderer.java	14 Jul 2010 20:21:23 -0000	1.10
>+++ src/org/eclipse/e4/ui/workbench/renderers/swt/SeparatorRenderer.java	19 Oct 2010 15:22:33 -0000
>@@ -12,11 +12,8 @@
> 
> import java.util.List;
> import org.eclipse.e4.ui.model.application.ui.MUIElement;
>-import org.eclipse.e4.ui.model.application.ui.menu.MMenuSeparator;
> import org.eclipse.e4.ui.model.application.ui.menu.MToolBarSeparator;
> import org.eclipse.swt.SWT;
>-import org.eclipse.swt.widgets.Menu;
>-import org.eclipse.swt.widgets.MenuItem;
> import org.eclipse.swt.widgets.ToolBar;
> import org.eclipse.swt.widgets.ToolItem;
> import org.eclipse.swt.widgets.Widget;
>@@ -41,21 +38,10 @@
> 			return null;
> 		}
> 		if (nextVisibleChild.isVisible()
>-				&& (nextVisibleChild instanceof MMenuSeparator || nextVisibleChild instanceof MToolBarSeparator)) {
>+				&& (nextVisibleChild instanceof MToolBarSeparator)) {
> 			return null;
> 		}
>-		if (element instanceof MMenuSeparator) {
>-			Menu menu = null;
>-			Object widget = element.getParent().getWidget();
>-			if (widget instanceof Menu) {
>-				menu = (Menu) widget;
>-			} else if (widget instanceof MenuItem) {
>-				menu = ((MenuItem) widget).getMenu();
>-			}
>-			if (menu != null) {
>-				newSep = new MenuItem(menu, SWT.SEPARATOR, addIndex);
>-			}
>-		} else if (element instanceof MToolBarSeparator) {
>+		if (element instanceof MToolBarSeparator) {
> 			ToolBar tb = parent instanceof ToolBar ? (ToolBar) parent
> 					: (ToolBar) element.getParent().getWidget();
> 			newSep = new ToolItem(tb, SWT.SEPARATOR, addIndex);
>Index: src/org/eclipse/e4/ui/workbench/renderers/swt/WorkbenchRendererFactory.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/WorkbenchRendererFactory.java,v
>retrieving revision 1.5
>diff -u -r1.5 WorkbenchRendererFactory.java
>--- src/org/eclipse/e4/ui/workbench/renderers/swt/WorkbenchRendererFactory.java	14 Oct 2010 18:21:18 -0000	1.5
>+++ src/org/eclipse/e4/ui/workbench/renderers/swt/WorkbenchRendererFactory.java	19 Oct 2010 15:22:33 -0000
>@@ -14,12 +14,8 @@
> import org.eclipse.e4.ui.model.application.ui.basic.MPartStack;
> import org.eclipse.e4.ui.model.application.ui.basic.MTrimBar;
> import org.eclipse.e4.ui.model.application.ui.basic.MWindow;
>-import org.eclipse.e4.ui.model.application.ui.menu.MDirectMenuItem;
>-import org.eclipse.e4.ui.model.application.ui.menu.MHandledMenuItem;
> import org.eclipse.e4.ui.model.application.ui.menu.MMenu;
> import org.eclipse.e4.ui.model.application.ui.menu.MMenuSeparator;
>-import org.eclipse.e4.ui.model.application.ui.menu.MRenderedMenu;
>-import org.eclipse.e4.ui.model.application.ui.menu.MRenderedMenuItem;
> import org.eclipse.e4.ui.model.application.ui.menu.MRenderedToolBar;
> import org.eclipse.e4.ui.model.application.ui.menu.MToolBar;
> import org.eclipse.e4.ui.model.application.ui.menu.MToolBarSeparator;
>@@ -30,8 +26,7 @@
> public class WorkbenchRendererFactory implements IRendererFactory {
> 
> 	private AreaRenderer areaRenderer;
>-	private MenuRenderer menuRenderer;
>-	private HandledMenuItemRenderer handledMenuItemRenderer;
>+	private MenuManagerRenderer menuRenderer;
> 	private ToolBarRenderer toolbarRenderer;
> 	private ToolItemRenderer toolItemRenderer;
> 	private SeparatorRenderer separatorRenderer;
>@@ -46,9 +41,6 @@
> 	private WBWRenderer wbwRenderer;
> 
> 	private IEclipseContext context;
>-	private DirectMenuItemRenderer directMenuItemRenderer;
>-	private RenderedMenuRenderer renderedMenuRenderer;
>-	private RenderedMenuItemRenderer renderedMenuItemRenderer;
> 	private RenderedToolBarRenderer renderedToolbarRenderer;
> 
> 	public AbstractPartRenderer getRenderer(MUIElement uiElement, Object parent) {
>@@ -64,33 +56,9 @@
> 				initRenderer(contributedPartRenderer);
> 			}
> 			return contributedPartRenderer;
>-		} else if (uiElement instanceof MHandledMenuItem) {
>-			if (handledMenuItemRenderer == null) {
>-				handledMenuItemRenderer = new HandledMenuItemRenderer();
>-				initRenderer(handledMenuItemRenderer);
>-			}
>-			return handledMenuItemRenderer;
>-		} else if (uiElement instanceof MDirectMenuItem) {
>-			if (directMenuItemRenderer == null) {
>-				directMenuItemRenderer = new DirectMenuItemRenderer();
>-				initRenderer(directMenuItemRenderer);
>-			}
>-			return directMenuItemRenderer;
>-		} else if (uiElement instanceof MRenderedMenu) {
>-			if (renderedMenuRenderer == null) {
>-				renderedMenuRenderer = new RenderedMenuRenderer();
>-				initRenderer(renderedMenuRenderer);
>-			}
>-			return renderedMenuRenderer;
>-		} else if (uiElement instanceof MRenderedMenuItem) {
>-			if (renderedMenuItemRenderer == null) {
>-				renderedMenuItemRenderer = new RenderedMenuItemRenderer();
>-				initRenderer(renderedMenuItemRenderer);
>-			}
>-			return renderedMenuItemRenderer;
> 		} else if (uiElement instanceof MMenu) {
> 			if (menuRenderer == null) {
>-				menuRenderer = new MenuRenderer();
>+				menuRenderer = new MenuManagerRenderer();
> 				initRenderer(menuRenderer);
> 			}
> 			return menuRenderer;
>#P org.eclipse.e4.ui.workbench.swt
>Index: src/org/eclipse/e4/ui/internal/workbench/swt/PartRenderingEngine.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench.swt/src/org/eclipse/e4/ui/internal/workbench/swt/PartRenderingEngine.java,v
>retrieving revision 1.50
>diff -u -r1.50 PartRenderingEngine.java
>--- src/org/eclipse/e4/ui/internal/workbench/swt/PartRenderingEngine.java	15 Oct 2010 17:58:23 -0000	1.50
>+++ src/org/eclipse/e4/ui/internal/workbench/swt/PartRenderingEngine.java	19 Oct 2010 15:22:34 -0000
>@@ -52,6 +52,7 @@
> import org.eclipse.e4.ui.model.application.ui.advanced.MPlaceholder;
> import org.eclipse.e4.ui.model.application.ui.basic.MTrimmedWindow;
> import org.eclipse.e4.ui.model.application.ui.basic.MWindow;
>+import org.eclipse.e4.ui.model.application.ui.menu.MMenu;
> import org.eclipse.e4.ui.services.IStylingEngine;
> import org.eclipse.e4.ui.workbench.IPresentationEngine;
> import org.eclipse.e4.ui.workbench.IResourceUtilities;
>@@ -59,7 +60,6 @@
> import org.eclipse.e4.ui.workbench.UIEvents;
> import org.eclipse.e4.ui.workbench.modeling.EModelService;
> import org.eclipse.e4.ui.workbench.swt.factories.IRendererFactory;
>-import org.eclipse.e4.ui.workbench.swt.modeling.MenuServiceFilter;
> import org.eclipse.emf.ecore.impl.EObjectImpl;
> import org.eclipse.equinox.app.IApplication;
> import org.eclipse.equinox.app.IApplicationContext;
>@@ -91,8 +91,6 @@
> 
> 	IRendererFactory curFactory = null;
> 
>-	MenuServiceFilter menuServiceFilter;
>-
> 	org.eclipse.swt.widgets.Listener keyListener;
> 
> 	// Life Cycle handlers
>@@ -109,8 +107,10 @@
> 						.eContainer();
> 			}
> 
>+			boolean menuChild = parent instanceof MMenu;
>+
> 			// If the parent isn't displayed who cares?
>-			if (parent == null || parent.getWidget() == null)
>+			if (parent == null || parent.getWidget() == null || menuChild)
> 				return;
> 
> 			if (changedElement.isToBeRendered()) {
>@@ -202,9 +202,10 @@
> 			MElementContainer<MUIElement> changedElement = (MElementContainer<MUIElement>) changedObj;
> 			boolean isApplication = changedObj instanceof MApplication;
> 
>+			boolean menuChild = changedObj instanceof MMenu;
> 			// If the parent isn't in the UI then who cares?
> 			AbstractPartRenderer renderer = getRendererFor(changedElement);
>-			if (!isApplication && renderer == null)
>+			if ((!isApplication && renderer == null) || menuChild)
> 				return;
> 
> 			String eventType = (String) event
>@@ -695,13 +696,6 @@
> 				display.addFilter(SWT.KeyDown, keyListener);
> 				display.addFilter(SWT.Traverse, keyListener);
> 
>-				menuServiceFilter = ContextInjectionFactory.make(
>-						MenuServiceFilter.class, runContext);
>-				display.addFilter(SWT.Show, menuServiceFilter);
>-				display.addFilter(SWT.Hide, menuServiceFilter);
>-				display.addFilter(SWT.Dispose, menuServiceFilter);
>-				runContext.set(MenuServiceFilter.class, menuServiceFilter);
>-
> 				// Show the initial UI
> 
> 				// Create a 'limbo' shell (used to host controls that shouldn't
>@@ -834,17 +828,6 @@
> 	 * why this is needed we should make this safe for multiple calls
> 	 */
> 	private void cleanUp() {
>-		if (menuServiceFilter != null) {
>-			Display display = Display.getDefault();
>-			if (!display.isDisposed()) {
>-				display.removeFilter(SWT.Show, menuServiceFilter);
>-				display.removeFilter(SWT.Hide, menuServiceFilter);
>-				display.removeFilter(SWT.Dispose, menuServiceFilter);
>-				menuServiceFilter.dispose();
>-				menuServiceFilter = null;
>-				appContext.remove(MenuServiceFilter.class);
>-			}
>-		}
> 		if (keyListener != null) {
> 			Display display = Display.getDefault();
> 			if (!display.isDisposed()) {
>Index: src/org/eclipse/e4/ui/workbench/swt/modeling/MenuManagerServiceFilter.java
>===================================================================
>RCS file: src/org/eclipse/e4/ui/workbench/swt/modeling/MenuManagerServiceFilter.java
>diff -N src/org/eclipse/e4/ui/workbench/swt/modeling/MenuManagerServiceFilter.java
>--- src/org/eclipse/e4/ui/workbench/swt/modeling/MenuManagerServiceFilter.java	14 Oct 2010 18:21:19 -0000	1.1
>+++ /dev/null	1 Jan 1970 00:00:00 -0000
>@@ -1,193 +0,0 @@
>-/*******************************************************************************
>- * Copyright (c) 2010 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
>- * http://www.eclipse.org/legal/epl-v10.html
>- *
>- * Contributors:
>- *     IBM Corporation - initial API and implementation
>- *******************************************************************************/
>-package org.eclipse.e4.ui.workbench.swt.modeling;
>-
>-import java.lang.reflect.Method;
>-import java.util.HashMap;
>-import javax.inject.Inject;
>-import org.eclipse.core.runtime.ISafeRunnable;
>-import org.eclipse.core.runtime.SafeRunner;
>-import org.eclipse.e4.core.contexts.IEclipseContext;
>-import org.eclipse.e4.core.services.log.Logger;
>-import org.eclipse.e4.ui.internal.workbench.swt.AbstractPartRenderer;
>-import org.eclipse.e4.ui.internal.workbench.swt.Policy;
>-import org.eclipse.e4.ui.internal.workbench.swt.WorkbenchSWTActivator;
>-import org.eclipse.e4.ui.model.application.MApplication;
>-import org.eclipse.e4.ui.model.application.ui.menu.MMenu;
>-import org.eclipse.e4.ui.model.application.ui.menu.MPopupMenu;
>-import org.eclipse.e4.ui.workbench.IPresentationEngine;
>-import org.eclipse.e4.ui.workbench.modeling.EModelService;
>-import org.eclipse.jface.action.MenuManager;
>-import org.eclipse.swt.SWT;
>-import org.eclipse.swt.widgets.Event;
>-import org.eclipse.swt.widgets.Listener;
>-import org.eclipse.swt.widgets.Menu;
>-import org.eclipse.swt.widgets.Widget;
>-
>-public class MenuManagerServiceFilter implements Listener {
>-	public static final String NUL_MENU_ITEM = "(None Applicable)"; //$NON-NLS-1$
>-
>-	private static final String TMP_ORIGINAL_CONTEXT = "MenuServiceFilter.original.context";
>-
>-	private static void trace(String msg, Widget menu, MMenu menuModel) {
>-		WorkbenchSWTActivator.trace(Policy.MENUS, msg + ": " + menu + ": "
>-				+ menuModel, null);
>-	}
>-
>-	private static Method aboutToShow;
>-
>-	public static Method getAboutToShow() {
>-		if (aboutToShow == null) {
>-			try {
>-				aboutToShow = MenuManager.class
>-						.getDeclaredMethod("handleAboutToShow");
>-				aboutToShow.setAccessible(true);
>-			} catch (SecurityException e) {
>-				// TODO Auto-generated catch block
>-				e.printStackTrace();
>-			} catch (NoSuchMethodException e) {
>-				// TODO Auto-generated catch block
>-				e.printStackTrace();
>-			}
>-		}
>-		return aboutToShow;
>-	}
>-
>-	@Inject
>-	private MApplication application;
>-
>-	@Inject
>-	private IPresentationEngine renderer;
>-
>-	@Inject
>-	private Logger logger;
>-
>-	@Inject
>-	EModelService modelService;
>-
>-	private HashMap<Menu, Runnable> pendingCleanup = new HashMap<Menu, Runnable>();
>-
>-	public void handleEvent(final Event event) {
>-		// wrap the handling in a SafeRunner so that exceptions do not prevent
>-		// the menu from being shown
>-		SafeRunner.run(new ISafeRunnable() {
>-			public void handleException(Throwable e) {
>-				if (e instanceof Error) {
>-					// errors are deadly, we shouldn't ignore these
>-					throw (Error) e;
>-				} else {
>-					// log exceptions otherwise
>-					if (logger != null) {
>-						logger.error(e);
>-					}
>-				}
>-			}
>-
>-			public void run() throws Exception {
>-				safeHandleEvent(event);
>-			}
>-		});
>-	}
>-
>-	private void safeHandleEvent(Event event) {
>-		if (!(event.widget instanceof Menu)) {
>-			return;
>-		}
>-		final Menu menu = (Menu) event.widget;
>-		if (event.type == SWT.Dispose) {
>-			trace("handleMenu.Dispose", menu, null);
>-			cleanUp(menu);
>-		}
>-		Object obj = menu.getData(AbstractPartRenderer.OWNING_ME);
>-		if (obj == null && menu.getParentItem() != null) {
>-			obj = menu.getParentItem().getData(AbstractPartRenderer.OWNING_ME);
>-		}
>-		if (obj instanceof MPopupMenu) {
>-			handleContextMenu(event, menu, (MPopupMenu) obj);
>-		} else if (obj instanceof MMenu) {
>-			handleMenu(event, menu, (MMenu) obj);
>-		}
>-	}
>-
>-	private void handleMenu(final Event event, final Menu menu,
>-			final MMenu menuModel) {
>-		if ((menu.getStyle() & SWT.BAR) != 0) {
>-			// don't process the menu bar, it's not fair :-)
>-			return;
>-		}
>-		switch (event.type) {
>-		case SWT.Show:
>-			cleanUp(menu);
>-			showMenu(event, menu, menuModel);
>-			break;
>-		case SWT.Hide:
>-			// TODO we'll clean up on show
>-			break;
>-		}
>-	}
>-
>-	public void showMenu(final Event event, final Menu menu,
>-			final MMenu menuModel) {
>-		// System.err.println("showMenu: " + menuModel + "\n\t" + menu);
>-	}
>-
>-	private void handleContextMenu(final Event event, final Menu menu,
>-			final MPopupMenu menuModel) {
>-		switch (event.type) {
>-		case SWT.Show:
>-			cleanUp(menu);
>-			showPopup(event, menu, menuModel);
>-			break;
>-		case SWT.Hide:
>-			hidePopup(event, menu, menuModel);
>-			break;
>-		}
>-	}
>-
>-	public void hidePopup(Event event, Menu menu, MPopupMenu menuModel) {
>-		// System.err.println("hidePopup: " + menuModel + "\n\t" + menu);
>-		final IEclipseContext popupContext = menuModel.getContext();
>-		final IEclipseContext originalChild = (IEclipseContext) popupContext
>-				.get(TMP_ORIGINAL_CONTEXT);
>-		popupContext.remove(TMP_ORIGINAL_CONTEXT);
>-		if (!menu.isDisposed()) {
>-			menu.getDisplay().asyncExec(new Runnable() {
>-				public void run() {
>-					if (originalChild == null) {
>-						popupContext.deactivate();
>-					} else {
>-						originalChild.activate();
>-					}
>-				}
>-			});
>-		}
>-	}
>-
>-	public void showPopup(final Event event, final Menu menu,
>-			final MPopupMenu menuModel) {
>-		// System.err.println("showPopup: " + menuModel + "\n\t" + menu);
>-		// we need some context foolery here
>-		final IEclipseContext popupContext = menuModel.getContext();
>-		final IEclipseContext parentContext = popupContext.getParent();
>-		final IEclipseContext originalChild = parentContext.getActiveChild();
>-		popupContext.activate();
>-		popupContext.set(TMP_ORIGINAL_CONTEXT, originalChild);
>-
>-	}
>-
>-	public void cleanUp(final Menu menu) {
>-		// System.err.println("cleanUp: " + menu);
>-	}
>-
>-	public void dispose() {
>-		// System.err.println("dispose");
>-	}
>-}
>Index: src/org/eclipse/e4/ui/workbench/swt/modeling/MenuService.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench.swt/src/org/eclipse/e4/ui/workbench/swt/modeling/MenuService.java,v
>retrieving revision 1.12
>diff -u -r1.12 MenuService.java
>--- src/org/eclipse/e4/ui/workbench/swt/modeling/MenuService.java	14 Oct 2010 18:21:19 -0000	1.12
>+++ src/org/eclipse/e4/ui/workbench/swt/modeling/MenuService.java	19 Oct 2010 15:22:34 -0000
>@@ -13,13 +13,13 @@
> import javax.inject.Inject;
> import org.eclipse.e4.core.contexts.IEclipseContext;
> import org.eclipse.e4.ui.internal.workbench.swt.AbstractPartRenderer;
>+import org.eclipse.e4.ui.model.application.ui.MElementContainer;
>+import org.eclipse.e4.ui.model.application.ui.MUIElement;
> import org.eclipse.e4.ui.model.application.ui.basic.MPart;
> import org.eclipse.e4.ui.model.application.ui.menu.MMenu;
> import org.eclipse.e4.ui.model.application.ui.menu.MPopupMenu;
>-import org.eclipse.swt.SWT;
>+import org.eclipse.e4.ui.workbench.swt.factories.IRendererFactory;
> import org.eclipse.swt.widgets.Control;
>-import org.eclipse.swt.widgets.Event;
>-import org.eclipse.swt.widgets.Listener;
> import org.eclipse.swt.widgets.Menu;
> 
> public class MenuService implements EMenuService {
>@@ -49,20 +49,39 @@
> 		if (mmenu.getWidget() != null) {
> 			return false;
> 		}
>-		Menu menu = new Menu(parentControl);
>-		parentControl.setMenu(menu);
>-		mmenu.setWidget(menu);
>-		menu.setData(AbstractPartRenderer.OWNING_ME, mmenu);
>+		// we need to delegate to the renderer so that it "processes" the
>+		// MenuManager correctly
>+		IRendererFactory rendererFactory = myPart.getContext().get(
>+				IRendererFactory.class);
>+		AbstractPartRenderer renderer = rendererFactory.getRenderer(mmenu,
>+				parentControl);
> 		IEclipseContext popupContext = myPart.getContext().createChild(
> 				"popup:" + mmenu.getElementId());
> 		mmenu.setContext(popupContext);
>-		menu.addListener(SWT.Dispose, new Listener() {
>-			public void handleEvent(Event event) {
>-				mmenu.getContext().dispose();
>-				mmenu.setContext(null);
>-				mmenu.setWidget(null);
>-			}
>-		});
>+		Object widget = renderer.createWidget(mmenu, parentControl);
>+		if (!(widget instanceof Menu)) {
>+			return false;
>+		}
>+		renderer.bindWidget(mmenu, widget);
>+		renderer.hookControllerLogic(mmenu);
>+
>+		// Process its internal structure through the renderer that created
>+		// it
>+		Object castObject = mmenu;
>+		renderer.processContents((MElementContainer<MUIElement>) castObject);
>+
>+		// Allow a final chance to set up
>+		renderer.postProcess(mmenu);
>+
>+		// Now that we have a widget let the parent (if any) know
>+		if (mmenu.getParent() instanceof MUIElement) {
>+			MElementContainer<MUIElement> parentElement = mmenu.getParent();
>+			AbstractPartRenderer parentRenderer = rendererFactory.getRenderer(
>+					parentElement, null);
>+			if (parentRenderer != null)
>+				parentRenderer.childRendered(parentElement, mmenu);
>+		}
>+
> 		return true;
> 	}
> }
>Index: src/org/eclipse/e4/ui/workbench/swt/modeling/MenuServiceFilter.java
>===================================================================
>RCS file: src/org/eclipse/e4/ui/workbench/swt/modeling/MenuServiceFilter.java
>diff -N src/org/eclipse/e4/ui/workbench/swt/modeling/MenuServiceFilter.java
>--- src/org/eclipse/e4/ui/workbench/swt/modeling/MenuServiceFilter.java	14 Oct 2010 18:21:19 -0000	1.30
>+++ /dev/null	1 Jan 1970 00:00:00 -0000
>@@ -1,369 +0,0 @@
>-/*******************************************************************************
>- * Copyright (c) 2010 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
>- * http://www.eclipse.org/legal/epl-v10.html
>- *
>- * Contributors:
>- *     IBM Corporation - initial API and implementation
>- *******************************************************************************/
>-package org.eclipse.e4.ui.workbench.swt.modeling;
>-
>-import java.lang.reflect.InvocationTargetException;
>-import java.lang.reflect.Method;
>-import java.util.ArrayList;
>-import java.util.HashMap;
>-import java.util.List;
>-import javax.inject.Inject;
>-import org.eclipse.core.commands.ParameterizedCommand;
>-import org.eclipse.core.runtime.ISafeRunnable;
>-import org.eclipse.core.runtime.SafeRunner;
>-import org.eclipse.e4.core.commands.EHandlerService;
>-import org.eclipse.e4.core.contexts.IEclipseContext;
>-import org.eclipse.e4.core.services.log.Logger;
>-import org.eclipse.e4.ui.internal.workbench.ContributionsAnalyzer;
>-import org.eclipse.e4.ui.internal.workbench.swt.AbstractPartRenderer;
>-import org.eclipse.e4.ui.internal.workbench.swt.Policy;
>-import org.eclipse.e4.ui.internal.workbench.swt.WorkbenchSWTActivator;
>-import org.eclipse.e4.ui.model.application.MApplication;
>-import org.eclipse.e4.ui.model.application.ui.menu.MHandledMenuItem;
>-import org.eclipse.e4.ui.model.application.ui.menu.MMenu;
>-import org.eclipse.e4.ui.model.application.ui.menu.MMenuContribution;
>-import org.eclipse.e4.ui.model.application.ui.menu.MMenuElement;
>-import org.eclipse.e4.ui.model.application.ui.menu.MPopupMenu;
>-import org.eclipse.e4.ui.model.application.ui.menu.MRenderedMenu;
>-import org.eclipse.e4.ui.workbench.IPresentationEngine;
>-import org.eclipse.e4.ui.workbench.modeling.EModelService;
>-import org.eclipse.e4.ui.workbench.modeling.ExpressionContext;
>-import org.eclipse.jface.action.MenuManager;
>-import org.eclipse.swt.SWT;
>-import org.eclipse.swt.widgets.Event;
>-import org.eclipse.swt.widgets.Listener;
>-import org.eclipse.swt.widgets.Menu;
>-import org.eclipse.swt.widgets.MenuItem;
>-import org.eclipse.swt.widgets.Widget;
>-
>-public class MenuServiceFilter implements Listener {
>-	public static final String NUL_MENU_ITEM = "(None Applicable)"; //$NON-NLS-1$
>-
>-	private static final String TMP_ORIGINAL_CONTEXT = "MenuServiceFilter.original.context";
>-
>-	private static void trace(String msg, Widget menu, MMenu menuModel) {
>-		WorkbenchSWTActivator.trace(Policy.MENUS, msg + ": " + menu + ": "
>-				+ menuModel, null);
>-	}
>-
>-	private static Method aboutToShow;
>-
>-	public static Method getAboutToShow() {
>-		if (aboutToShow == null) {
>-			try {
>-				aboutToShow = MenuManager.class
>-						.getDeclaredMethod("handleAboutToShow");
>-				aboutToShow.setAccessible(true);
>-			} catch (SecurityException e) {
>-				// TODO Auto-generated catch block
>-				e.printStackTrace();
>-			} catch (NoSuchMethodException e) {
>-				// TODO Auto-generated catch block
>-				e.printStackTrace();
>-			}
>-		}
>-		return aboutToShow;
>-	}
>-
>-	@Inject
>-	private MApplication application;
>-
>-	@Inject
>-	private IPresentationEngine renderer;
>-
>-	@Inject
>-	private Logger logger;
>-
>-	@Inject
>-	EModelService modelService;
>-
>-	private HashMap<Menu, Runnable> pendingCleanup = new HashMap<Menu, Runnable>();
>-
>-	public void handleEvent(final Event event) {
>-		// wrap the handling in a SafeRunner so that exceptions do not prevent
>-		// the menu from being shown
>-		SafeRunner.run(new ISafeRunnable() {
>-			public void handleException(Throwable e) {
>-				if (e instanceof Error) {
>-					// errors are deadly, we shouldn't ignore these
>-					throw (Error) e;
>-				} else {
>-					// log exceptions otherwise
>-					if (logger != null) {
>-						logger.error(e);
>-					}
>-				}
>-			}
>-
>-			public void run() throws Exception {
>-				safeHandleEvent(event);
>-			}
>-		});
>-	}
>-
>-	private void safeHandleEvent(Event event) {
>-		if (!(event.widget instanceof Menu)) {
>-			return;
>-		}
>-		final Menu menu = (Menu) event.widget;
>-		if (event.type == SWT.Dispose) {
>-			trace("handleMenu.Dispose", menu, null);
>-			cleanUp(menu);
>-		}
>-		Object obj = menu.getData(AbstractPartRenderer.OWNING_ME);
>-		if (obj == null && menu.getParentItem() != null) {
>-			obj = menu.getParentItem().getData(AbstractPartRenderer.OWNING_ME);
>-		}
>-		if (obj instanceof MRenderedMenu) {
>-			handlerRenderedMenu(event, menu, (MRenderedMenu) obj);
>-		} else if (obj instanceof MPopupMenu) {
>-			handleContextMenu(event, menu, (MPopupMenu) obj);
>-		} else if (obj instanceof MMenu) {
>-			handleMenu(event, menu, (MMenu) obj);
>-		}
>-	}
>-
>-	private void handleMenu(final Event event, final Menu menu,
>-			final MMenu menuModel) {
>-		if ((menu.getStyle() & SWT.BAR) != 0) {
>-			// don't process the menu bar, it's not fair :-)
>-			return;
>-		}
>-		switch (event.type) {
>-		case SWT.Show:
>-			trace("handleMenu.Show", menu, menuModel);
>-			cleanUp(menu);
>-			showMenu(event, menu, menuModel);
>-			break;
>-		case SWT.Hide:
>-			trace("handleMenu.Hide", menu, menuModel);
>-			// TODO we'll clean up on show
>-			break;
>-		}
>-	}
>-
>-	public void showMenu(final Event event, final Menu menu,
>-			final MMenu menuModel) {
>-		final IEclipseContext parentContext = modelService
>-				.getContainingContext(menuModel);
>-
>-		final ArrayList<MMenuContribution> toContribute = new ArrayList<MMenuContribution>();
>-		final ArrayList<MMenuElement> menuContributionsToRemove = new ArrayList<MMenuElement>();
>-		ExpressionContext eContext = new ExpressionContext(parentContext);
>-		ContributionsAnalyzer.gatherMenuContributions(menuModel,
>-				application.getMenuContributions(), menuModel.getElementId(),
>-				toContribute, eContext, false);
>-		if (menu.getItemCount() == 1) {
>-			MenuItem item = menu.getItem(0);
>-			if (NUL_MENU_ITEM.equals(item.getText())) {
>-				item.dispose();
>-			}
>-		}
>-		ContributionsAnalyzer.addMenuContributions(menuModel, toContribute,
>-				menuContributionsToRemove);
>-
>-		// create a cleanup routine for the Hide or next Show
>-		pendingCleanup.put(menu, new Runnable() {
>-			public void run() {
>-				if (!menu.isDisposed()) {
>-					unrender(menuContributionsToRemove);
>-				}
>-				removeMenuContributions(menuModel, menuContributionsToRemove);
>-			}
>-		});
>-		render(menu, menuModel);
>-		if (menu.getItemCount() == 0) {
>-			MenuItem menuItem = new MenuItem(menu, SWT.PUSH);
>-			menuItem.setText(NUL_MENU_ITEM);
>-			menuItem.setEnabled(false);
>-		}
>-	}
>-
>-	private void handleContextMenu(final Event event, final Menu menu,
>-			final MPopupMenu menuModel) {
>-		switch (event.type) {
>-		case SWT.Show:
>-			trace("handleContextMenu.Show", menu, menuModel);
>-			cleanUp(menu);
>-			showPopup(event, menu, menuModel);
>-			break;
>-		case SWT.Hide:
>-			trace("handleContextMenu.Hide", menu, menuModel);
>-			hidePopup(event, menu, menuModel);
>-			break;
>-		}
>-	}
>-
>-	public void hidePopup(Event event, Menu menu, MPopupMenu menuModel) {
>-		final IEclipseContext popupContext = menuModel.getContext();
>-		final IEclipseContext originalChild = (IEclipseContext) popupContext
>-				.get(TMP_ORIGINAL_CONTEXT);
>-		popupContext.remove(TMP_ORIGINAL_CONTEXT);
>-		if (!menu.isDisposed()) {
>-			menu.getDisplay().asyncExec(new Runnable() {
>-				public void run() {
>-					if (originalChild == null) {
>-						popupContext.deactivate();
>-					} else {
>-						originalChild.activate();
>-					}
>-				}
>-			});
>-		}
>-	}
>-
>-	public void showPopup(final Event event, final Menu menu,
>-			final MPopupMenu menuModel) {
>-		// System.err.println("showPopup: " + menuModel + "\n\t" + menu);
>-		// we need some context foolery here
>-		final IEclipseContext popupContext = menuModel.getContext();
>-		final IEclipseContext parentContext = popupContext.getParent();
>-		final IEclipseContext originalChild = parentContext.getActiveChild();
>-		popupContext.activate();
>-		popupContext.set(TMP_ORIGINAL_CONTEXT, originalChild);
>-
>-		final ArrayList<MMenuContribution> toContribute = new ArrayList<MMenuContribution>();
>-		final ArrayList<MMenuElement> menuContributionsToRemove = new ArrayList<MMenuElement>();
>-		ExpressionContext eContext = new ExpressionContext(popupContext);
>-		ContributionsAnalyzer.gatherMenuContributions(menuModel,
>-				application.getMenuContributions(), menuModel.getElementId(),
>-				toContribute, eContext, true);
>-
>-		for (String tag : menuModel.getTags()) {
>-			if (tag.startsWith("popup:") && tag.length() > 6) {
>-				ContributionsAnalyzer.gatherMenuContributions(menuModel,
>-						application.getMenuContributions(), tag.substring(6),
>-						toContribute, eContext, false);
>-			}
>-		}
>-		ContributionsAnalyzer.addMenuContributions(menuModel, toContribute,
>-				menuContributionsToRemove);
>-
>-		// create a cleanup routine for the Hide or next Show
>-		pendingCleanup.put(menu, new Runnable() {
>-			public void run() {
>-				if (!menu.isDisposed()) {
>-					unrender(menuContributionsToRemove);
>-				}
>-				removeMenuContributions(menuModel, menuContributionsToRemove);
>-			}
>-		});
>-		render(menu, menuModel);
>-	}
>-
>-	private void render(final Menu menu, final MMenu menuModel) {
>-		trace("render", menu, menuModel);
>-		for (MMenuElement element : menuModel.getChildren()) {
>-			renderer.createGui(element, menu, null);
>-			if (element instanceof MHandledMenuItem) {
>-				setEnabled((MHandledMenuItem) element);
>-			}
>-		}
>-	}
>-
>-	private void setEnabled(MHandledMenuItem item) {
>-		if (!item.isToBeRendered() || !item.isVisible()
>-				|| item.getWidget() == null) {
>-			return;
>-		}
>-		ParameterizedCommand cmd = item.getWbCommand();
>-		if (cmd == null) {
>-			return;
>-		}
>-		final IEclipseContext lclContext = modelService
>-				.getContainingContext(item);
>-		EHandlerService service = lclContext.get(EHandlerService.class);
>-		item.setEnabled(service.canExecute(cmd));
>-	}
>-
>-	private void unrender(final List<MMenuElement> menuModel) {
>-		trace("unrender", null, null);
>-		for (MMenuElement element : menuModel) {
>-			renderer.removeGui(element);
>-		}
>-	}
>-
>-	private void removeMenuContributions(final MMenu menuModel,
>-			final ArrayList<MMenuElement> menuContributionsToRemove) {
>-		for (MMenuElement item : menuContributionsToRemove) {
>-			trace("removeMenuContributions " + item,
>-					(Widget) menuModel.getWidget(), menuModel);
>-			menuModel.getChildren().remove(item);
>-		}
>-	}
>-
>-	private void handlerRenderedMenu(final Event event, final Menu menu,
>-			final MRenderedMenu menuModel) {
>-		// Do nothing here for the moment, except process any cleanups
>-		switch (event.type) {
>-		case SWT.Show:
>-			trace("handlerRenderedMenu.Show", menu, menuModel);
>-			cleanUp(menu);
>-			showRenderedMenu(event, menu, menuModel);
>-			break;
>-		case SWT.Hide:
>-			trace("handlerRenderedMenu.Hide", menu, menuModel);
>-			// TODO don't care
>-			break;
>-		}
>-	}
>-
>-	public void showRenderedMenu(final Event event, final Menu menu,
>-			final MRenderedMenu menuModel) {
>-		if (!(menuModel.getContributionManager() instanceof MenuManager)) {
>-			return;
>-		}
>-
>-		MenuManager manager = (MenuManager) menuModel.getContributionManager();
>-		Method handleAboutToShow = getAboutToShow();
>-		try {
>-			handleAboutToShow.invoke(manager);
>-		} catch (IllegalArgumentException e) {
>-			// TODO Auto-generated catch block
>-			e.printStackTrace();
>-		} catch (IllegalAccessException e) {
>-			// TODO Auto-generated catch block
>-			e.printStackTrace();
>-		} catch (InvocationTargetException e) {
>-			// TODO Auto-generated catch block
>-			e.printStackTrace();
>-		}
>-
>-		if (menuModel.getChildren().size() == 1
>-				&& menuModel.getChildren().get(0) instanceof MPopupMenu) {
>-			showPopup(event, menu, (MPopupMenu) menuModel.getChildren().get(0));
>-		} else {
>-			showMenu(event, menu, menuModel);
>-		}
>-		event.type = SWT.None;
>-		event.doit = false;
>-	}
>-
>-	public void cleanUp(final Menu menu) {
>-		trace("cleanUp", menu, null);
>-		if (pendingCleanup.isEmpty()) {
>-			return;
>-		}
>-		Runnable cleanUp = pendingCleanup.remove(menu);
>-		if (cleanUp != null) {
>-			trace("cleanUp.run()", menu, null);
>-			cleanUp.run();
>-		}
>-	}
>-
>-	public void dispose() {
>-		Menu[] keys = pendingCleanup.keySet().toArray(
>-				new Menu[pendingCleanup.size()]);
>-		for (Menu menu : keys) {
>-			cleanUp(menu);
>-		}
>-	}
>-}
>#P org.eclipse.ui.workbench
>Index: Eclipse UI/org/eclipse/ui/internal/e4/compatibility/CompatibilityView.java
>===================================================================
>RCS file: /cvsroot/eclipse/e4/org.eclipse.e4.compatibility/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/e4/compatibility/CompatibilityView.java,v
>retrieving revision 1.34
>diff -u -r1.34 CompatibilityView.java
>--- Eclipse UI/org/eclipse/ui/internal/e4/compatibility/CompatibilityView.java	14 Oct 2010 18:21:19 -0000	1.34
>+++ Eclipse UI/org/eclipse/ui/internal/e4/compatibility/CompatibilityView.java	19 Oct 2010 15:22:34 -0000
>@@ -12,14 +12,18 @@
> package org.eclipse.ui.internal.e4.compatibility;
> 
> import javax.inject.Inject;
>+import org.eclipse.e4.core.contexts.IEclipseContext;
> import org.eclipse.e4.ui.internal.workbench.ContributionsAnalyzer;
>+import org.eclipse.e4.ui.internal.workbench.swt.AbstractPartRenderer;
> import org.eclipse.e4.ui.model.application.ui.basic.MPart;
> import org.eclipse.e4.ui.model.application.ui.menu.MMenu;
>-import org.eclipse.e4.ui.model.application.ui.menu.MRenderedMenu;
> import org.eclipse.e4.ui.model.application.ui.menu.MRenderedToolBar;
> import org.eclipse.e4.ui.model.application.ui.menu.MToolBar;
> import org.eclipse.e4.ui.model.application.ui.menu.impl.MenuFactoryImpl;
>+import org.eclipse.e4.ui.workbench.renderers.swt.MenuManagerRenderer;
>+import org.eclipse.e4.ui.workbench.renderers.swt.MenuRenderer;
> import org.eclipse.e4.ui.workbench.renderers.swt.StackRenderer;
>+import org.eclipse.e4.ui.workbench.swt.factories.IRendererFactory;
> import org.eclipse.jface.action.MenuManager;
> import org.eclipse.jface.action.ToolBarManager;
> import org.eclipse.swt.widgets.Composite;
>@@ -75,16 +79,19 @@
> 		// dispose the tb, it will be re-created when the tab is shown
> 		tb.dispose();
> 
>+		IEclipseContext context = getModel().getContext();
>+		IRendererFactory rendererFactory = context.get(IRendererFactory.class);
>+
> 		MenuManager mm = (MenuManager) actionBars.getMenuManager();
>-		MRenderedMenu menu = null;
>+		MMenu menu = null;
> 		for (MMenu me : part.getMenus()) {
>-			if (me.getTags().contains(StackRenderer.TAG_VIEW_MENU) && (me instanceof MRenderedMenu)) {
>-				menu = (MRenderedMenu) me;
>+			if (me.getTags().contains(StackRenderer.TAG_VIEW_MENU)) {
>+				menu = me;
> 				break;
> 			}
> 		}
> 		if (menu == null) {
>-			menu = MenuFactoryImpl.eINSTANCE.createRenderedMenu();
>+			menu = MenuFactoryImpl.eINSTANCE.createMenu();
> 			menu.setElementId(part.getElementId());
> 
> 			menu.getTags().add(StackRenderer.TAG_VIEW_MENU);
>@@ -92,7 +99,10 @@
> 			part.getMenus().add(menu);
> 
> 		}
>-		menu.setContributionManager(mm);
>+		AbstractPartRenderer apr = rendererFactory.getRenderer(menu, parent);
>+		if (apr instanceof MenuRenderer) {
>+			((MenuManagerRenderer) apr).linkModelToManager(menu, mm);
>+		}
> 
> 		// Construct the toolbar (if necessary)
> 		MToolBar toolbar = part.getToolbar();

Actions: View | Diff

Attachments on bug 325392: 179140 | 180268 | 180514 | 180901 | 181172 | 181191 | 181226 | 182202 | 182509 | 182904 | 183010