Implement activity support filtering for e4 dynamic menu contribution.

Similar to PluginActionContributionItem we can support activity
filtering of menu item to be shown or not.

see #2217

Author: raghucssit

bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/DynamicContributionContributionItem.java

@@ -14,11 +14,16 @@
 
 package org.eclipse.e4.ui.workbench.renderers.swt;
 
+import org.eclipse.core.runtime.Platform;
+import org.eclipse.e4.ui.activitysupport.IActivityManagerProxy;
 import org.eclipse.e4.ui.model.application.ui.menu.MDynamicMenuContribution;
 import org.eclipse.jface.action.ContributionItem;
 import org.eclipse.jface.action.IContributionManager;
 import org.eclipse.jface.action.IMenuListener;
 import org.eclipse.jface.action.IMenuManager;
+import org.osgi.framework.BundleContext;
+import org.osgi.framework.FrameworkUtil;
+import org.osgi.util.tracker.ServiceTracker;
 
 /**
  * This item currently serves as a placeholder to determine the correct location
@@ -29,12 +34,17 @@ class DynamicContributionContributionItem extends ContributionItem {
 
 	private IMenuListener menuListener = IMenuManager::markDirty;
 
+	private static final String BUNDLE_CLASS_PREFIX = "bundleclass://"; //$NON-NLS-1$
+
+	IActivityManagerProxy activitySupportProxy;
+
 	/**
 	 * Create the item and associated model;
 	 */
 	public DynamicContributionContributionItem(MDynamicMenuContribution item) {
 		super(item.getElementId());
 		model = item;
+		initializeAcivitySupportProxy();
 	}
 
 	@Override
@@ -66,4 +76,34 @@ public void setParent(IContributionManager parent) {
 		}
 		super.setParent(parent);
 	}
+
+	@Override
+	public boolean isVisible() {
+		if (this.activitySupportProxy != null) {
+			// Contribution URI has the scheme bundleclass://. Ex:
+			// bundleclass://org.eclipse.pde.spy.core/org.eclipse.pde.spy.core.SpyProcessor
+			String contributionURI = this.getModel().getContributionURI();
+			if (contributionURI.startsWith(BUNDLE_CLASS_PREFIX)) {
+				return activitySupportProxy
+						.isIdentifierEnabled(contributionURI.substring(BUNDLE_CLASS_PREFIX.length()));
+			}
+		}
+		return true;
+	}
+
+	/**
+	 * Initialize the Activity Support proxy from Platform Context
+	 */
+	private void initializeAcivitySupportProxy() {
+		BundleContext context = FrameworkUtil.getBundle(Platform.class).getBundleContext();
+		if (context != null) {
+			ServiceTracker<IActivityManagerProxy, IActivityManagerProxy> tracker = new ServiceTracker<>(context,
+					IActivityManagerProxy.class, null);
+			if (tracker != null) {
+				tracker.open();
+				this.activitySupportProxy = tracker.getService();
+				tracker.close();
+			}
+		}
+	}
 }

bundles/org.eclipse.e4.ui.workbench/META-INF/MANIFEST.MF

@@ -20,7 +20,8 @@ Require-Bundle: org.eclipse.e4.ui.model.workbench;bundle-version="1.2.0",
  org.eclipse.core.runtime;bundle-version="[3.29.0,4.0.0)"
 Bundle-ActivationPolicy: lazy
 Bundle-RequiredExecutionEnvironment: JavaSE-17
-Export-Package: org.eclipse.e4.ui.internal.workbench;
+Export-Package: org.eclipse.e4.ui.activitysupport,
+ org.eclipse.e4.ui.internal.workbench;
   x-friends:="org.eclipse.e4.ui.workbench.fragment,
    org.eclipse.e4.ui.workbench.addons.swt,
    org.eclipse.e4.ui.workbench.renderers.swt,

bundles/org.eclipse.e4.ui.workbench/src/org/eclipse/e4/ui/activitysupport/IActivityManagerProxy.java

@@ -0,0 +1,38 @@
+/*******************************************************************************
+ * Copyright (c) 2024 Advantest Europe GmbH and others.
+ *
+ * This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License 2.0
+ * which accompanies this distribution, and is available at
+ * https://www.eclipse.org/legal/epl-2.0/
+ *
+ * SPDX-License-Identifier: EPL-2.0
+ *
+ * Contributors:
+ * 				Raghunandana Murthappa
+ *******************************************************************************/
+package org.eclipse.e4.ui.activitysupport;
+
+/**
+ * A bridge between org.eclipse.ui.workbench and
+ * org.eclipse.e4.ui.workbench.renderers.swt.
+ *
+ * Service for this interface is bound to Platform.class bundle at
+ * Workbench.class. We cannot depend on org.eclipse.ui.workbench from
+ * org.eclipse.e4.ui.workbench.renderers.swt
+ *
+ * @since 1.16
+ */
+public interface IActivityManagerProxy {
+	/**
+	 * Checks whether the given element is enabled or not in the workbench activity
+	 * support.
+	 *
+	 * @param identifierId A qualified id if the contribution. Which has format of
+	 *                     bundle-id/element. Ex:
+	 *                     org.eclipse.pde.spy.core/org.eclipse.pde.spy.core.SpyProcessor
+	 * @return {@code true} if the given identifierId is enabled within workbench
+	 *         activity support.
+	 */
+	public boolean isIdentifierEnabled(String identifierId);
+}

bundles/org.eclipse.ui.workbench/META-INF/MANIFEST.MF

@@ -113,7 +113,8 @@ Require-Bundle: org.eclipse.core.runtime;bundle-version="[3.29.0,4.0.0)",
  org.eclipse.e4.ui.workbench.addons.swt;bundle-version="0.10.0",
  org.eclipse.e4.ui.services;bundle-version="1.3.0",
  org.eclipse.emf.ecore.xmi;bundle-version="2.11.0",
- org.eclipse.e4.core.di.extensions;bundle-version="0.13.0"
+ org.eclipse.e4.core.di.extensions;bundle-version="0.13.0",
+ org.eclipse.e4.ui.workbench;bundle-version="1.16.0"
 Import-Package: com.ibm.icu.util,
  jakarta.annotation;version="[2.1.0,3.0.0)",
  jakarta.inject;version="[2.0.0,3.0.0)",

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/Workbench.java

@@ -86,6 +86,7 @@
 import org.eclipse.e4.core.contexts.IEclipseContext;
 import org.eclipse.e4.core.di.InjectionException;
 import org.eclipse.e4.core.services.events.IEventBroker;
+import org.eclipse.e4.ui.activitysupport.IActivityManagerProxy;
 import org.eclipse.e4.ui.internal.workbench.E4Workbench;
 import org.eclipse.e4.ui.internal.workbench.E4XMIResource;
 import org.eclipse.e4.ui.internal.workbench.renderers.swt.IUpdateService;
@@ -245,6 +246,7 @@
 import org.eclipse.ui.internal.themes.FontDefinition;
 import org.eclipse.ui.internal.themes.ThemeElementHelper;
 import org.eclipse.ui.internal.themes.WorkbenchThemeManager;
+import org.eclipse.ui.internal.util.ActivityManagerProxy;
 import org.eclipse.ui.internal.util.PrefUtil;
 import org.eclipse.ui.intro.IIntroManager;
 import org.eclipse.ui.keys.IBindingService;
@@ -2206,6 +2208,11 @@ public Object compute(IEclipseContext context, String contextKey) {
 			}
 		});
 		WorkbenchPlugin.getDefault().initializeContext(e4Context);
+
+		BundleContext context = FrameworkUtil.getBundle(Platform.class).getBundleContext();
+		IWorkbenchActivitySupport activitySupport = PlatformUI.getWorkbench().getActivitySupport();
+		ActivityManagerProxy service = new ActivityManagerProxy(activitySupport);
+		context.registerService(IActivityManagerProxy.class.getName(), service, null);
 	}
 
 	private ArrayList<MCommand> commandsToRemove = new ArrayList<>();

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/util/ActivityManagerProxy.java

@@ -0,0 +1,32 @@
+/*******************************************************************************
+ * Copyright (c) 2024 Advantest Europe GmbH and others.
+ *
+ * This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License 2.0
+ * which accompanies this distribution, and is available at
+ * https://www.eclipse.org/legal/epl-2.0/
+ *
+ * SPDX-License-Identifier: EPL-2.0
+ *
+ * Contributors:
+ * 				Raghunandana Murthappa
+ *******************************************************************************/
+package org.eclipse.ui.internal.util;
+
+import org.eclipse.e4.ui.activitysupport.IActivityManagerProxy;
+import org.eclipse.ui.activities.IWorkbenchActivitySupport;
+
+public class ActivityManagerProxy implements IActivityManagerProxy {
+
+	private final IWorkbenchActivitySupport wbActivitySupport;
+
+	public ActivityManagerProxy(IWorkbenchActivitySupport wbActivitySupport) {
+		this.wbActivitySupport = wbActivitySupport;
+	}
+
+	@Override
+	public boolean isIdentifierEnabled(String identifierId) {
+		boolean isEnabled = wbActivitySupport.getActivityManager().getIdentifier(identifierId).isEnabled();
+		return isEnabled;
+	}
+}