Clean Code for bundles/org.eclipse.ui.workbench

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/IKeyBindingService.java

@@ -43,26 +43,30 @@ public interface IKeyBindingService {
 	 *
 	 * @return the active accelerator scope ids.
 	 */
+	@Deprecated
 	String[] getScopes();
 
 	/**
 	 * Registers an action with the key binding service.
 	 *
 	 * @param action the action to be registered with the key binding service.
 	 */
+	@Deprecated
 	void registerAction(IAction action);
 
 	/**
 	 * Sets the active accelerator scope ids.
 	 *
 	 * @param scopes the active accelerator scope ids.
 	 */
+	@Deprecated
 	void setScopes(String[] scopes);
 
 	/**
 	 * Unregisters an action with the key binding service.
 	 *
 	 * @param action the action to be unregistered with the key binding service.
 	 */
+	@Deprecated
 	void unregisterAction(IAction action);
 }

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/INestableKeyBindingService.java

@@ -46,6 +46,7 @@ public interface INestableKeyBindingService extends IKeyBindingService {
 	 *         case of a <code>null</code> parameter); <code>false</code> if nothing
 	 *         changed.
 	 */
+	@Deprecated
 	boolean activateKeyBindingService(IWorkbenchSite nestedSite);
 
 	/**
@@ -58,6 +59,7 @@ public interface INestableKeyBindingService extends IKeyBindingService {
 	 * @return The associated service, if any; or a new associated service, if none
 	 *         existed previously.
 	 */
+	@Deprecated
 	IKeyBindingService getKeyBindingService(IWorkbenchSite nestedSite);
 
 	/**
@@ -70,6 +72,7 @@ public interface INestableKeyBindingService extends IKeyBindingService {
 	 * @return <code>true</code> if the service existed and could be removed;
 	 *         <code>false</code> otherwise.
 	 */
+	@Deprecated
 	boolean removeKeyBindingService(IWorkbenchSite nestedSite);
 
 }

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/actions/OpenNewPageMenu.java

@@ -63,6 +63,7 @@ public class OpenNewPageMenu extends PerspectiveMenu {
 	 * @param window the window where a new page is created if an item within the
 	 *               menu is selected
 	 */
+	@Deprecated
 	public OpenNewPageMenu(IWorkbenchWindow window) {
 		this(window, null);
 	}
@@ -74,11 +75,13 @@ public OpenNewPageMenu(IWorkbenchWindow window) {
 	 *               menu is selected
 	 * @param input  the page input
 	 */
+	@Deprecated
 	public OpenNewPageMenu(IWorkbenchWindow window, IAdaptable input) {
 		super(window, "Open New Page Menu");//$NON-NLS-1$
 		this.pageInput = input;
 	}
 
+	@Deprecated
 	@Override
 	protected void run(IPerspectiveDescriptor desc) {
 		// Verify page input.
@@ -102,6 +105,7 @@ protected void run(IPerspectiveDescriptor desc) {
 	 *
 	 * @param input the page input
 	 */
+	@Deprecated
 	public void setPageInput(IAdaptable input) {
 		pageInput = input;
 	}

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/actions/OpenNewWindowMenu.java

@@ -62,6 +62,7 @@ public class OpenNewWindowMenu extends PerspectiveMenu {
 	 * @param window the window where a new page is created if an item within the
 	 *               menu is selected
 	 */
+	@Deprecated
 	public OpenNewWindowMenu(IWorkbenchWindow window) {
 		this(window, null);
 	}
@@ -73,6 +74,7 @@ public OpenNewWindowMenu(IWorkbenchWindow window) {
 	 *               menu is selected
 	 * @param input  the page input
 	 */
+	@Deprecated
 	public OpenNewWindowMenu(IWorkbenchWindow window, IAdaptable input) {
 		super(window, "Open New Page Menu");//$NON-NLS-1$
 		this.pageInput = input;
@@ -81,6 +83,7 @@ public OpenNewWindowMenu(IWorkbenchWindow window, IAdaptable input) {
 	/**
 	 * Opens a new window with a particular perspective and input.
 	 */
+	@Deprecated
 	@Override
 	protected void run(IPerspectiveDescriptor desc) {
 		// Verify page input.
@@ -104,6 +107,7 @@ protected void run(IPerspectiveDescriptor desc) {
 	 *
 	 * @param input the page input
 	 */
+	@Deprecated
 	public void setPageInput(IAdaptable input) {
 		pageInput = input;
 	}

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/actions/OpenPerspectiveMenu.java

@@ -70,6 +70,7 @@ public class OpenPerspectiveMenu extends PerspectiveMenu {
 	 * @param window      the window where a new page is created if an item within
 	 *                    the menu is selected
 	 */
+	@Deprecated
 	public OpenPerspectiveMenu(IMenuManager menuManager, IWorkbenchWindow window) {
 		this(window);
 		this.parentMenuManager = menuManager;
@@ -88,6 +89,7 @@ public OpenPerspectiveMenu(IMenuManager menuManager, IWorkbenchWindow window) {
 	 * @param window the window where a new page is created if an item within the
 	 *               menu is selected
 	 */
+	@Deprecated
 	public OpenPerspectiveMenu(IWorkbenchWindow window) {
 		this(window, null);
 		showActive(true);
@@ -100,6 +102,7 @@ public OpenPerspectiveMenu(IWorkbenchWindow window) {
 	 *               menu is selected
 	 * @param input  the page input
 	 */
+	@Deprecated
 	public OpenPerspectiveMenu(IWorkbenchWindow window, IAdaptable input) {
 		super(window, "Open New Page Menu");//$NON-NLS-1$
 		this.pageInput = input;
@@ -129,6 +132,7 @@ private String openPerspectiveSetting() {
 	 *
 	 * @param desc the selected perspective
 	 */
+	@Deprecated
 	@Override
 	protected void run(IPerspectiveDescriptor desc) {
 		openPage(desc);
@@ -140,6 +144,7 @@ protected void run(IPerspectiveDescriptor desc) {
 	 * @param desc  the selected perspective
 	 * @param event the event sent along with the selection callback
 	 */
+	@Deprecated
 	@Override
 	protected void run(IPerspectiveDescriptor desc, SelectionEvent event) {
 		openPage(desc);
@@ -166,13 +171,15 @@ private void openPage(IPerspectiveDescriptor desc) {
 	 *
 	 * @param input the page input
 	 */
+	@Deprecated
 	public void setPageInput(IAdaptable input) {
 		pageInput = input;
 	}
 
 	/**
 	 * @param isEnabled true if replace menu item is enabled within its parent menu.
 	 */
+	@Deprecated
 	public void setReplaceEnabled(boolean isEnabled) {
 		if (replaceEnabled != isEnabled) {
 			replaceEnabled = isEnabled;

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/contexts/ContextEvent.java

@@ -63,6 +63,7 @@ public final class ContextEvent {
 	 * @param nameChanged     true, iff the name property changed.
 	 * @param parentIdChanged true, iff the parentId property changed.
 	 */
+	@Deprecated
 	public ContextEvent(IContext context, boolean definedChanged, boolean enabledChanged, boolean nameChanged,
 			boolean parentIdChanged) {
 		if (context == null) {
@@ -82,6 +83,7 @@ public ContextEvent(IContext context, boolean definedChanged, boolean enabledCha
 	 * @return the instance of the interface that changed. Guaranteed not to be
 	 *         <code>null</code>.
 	 */
+	@Deprecated
 	public IContext getContext() {
 		return context;
 	}
@@ -91,6 +93,7 @@ public IContext getContext() {
 	 *
 	 * @return true, iff the defined property changed.
 	 */
+	@Deprecated
 	public boolean hasDefinedChanged() {
 		return definedChanged;
 	}
@@ -100,6 +103,7 @@ public boolean hasDefinedChanged() {
 	 *
 	 * @return true, iff the enabled property changed.
 	 */
+	@Deprecated
 	public boolean hasEnabledChanged() {
 		return enabledChanged;
 	}
@@ -109,6 +113,7 @@ public boolean hasEnabledChanged() {
 	 *
 	 * @return true, iff the name property changed.
 	 */
+	@Deprecated
 	public boolean hasNameChanged() {
 		return nameChanged;
 	}
@@ -118,6 +123,7 @@ public boolean hasNameChanged() {
 	 *
 	 * @return true, iff the parentId property changed.
 	 */
+	@Deprecated
 	public boolean hasParentIdChanged() {
 		return parentIdChanged;
 	}

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/contexts/ContextException.java

@@ -41,6 +41,7 @@ public abstract class ContextException extends Exception {
 	 *
 	 * @param message the detail message.
 	 */
+	@Deprecated
 	public ContextException(String message) {
 		super(message);
 	}
@@ -52,6 +53,7 @@ public ContextException(String message) {
 	 * @param message the detail message.
 	 * @param cause   the cause.
 	 */
+	@Deprecated
 	public ContextException(String message, Throwable cause) {
 		super(message);
 		// don't pass the cause to super, to allow compilation against JCL Foundation
@@ -65,6 +67,7 @@ public ContextException(String message, Throwable cause) {
 	 * @return the cause or <code>null</code>
 	 * @since 3.1
 	 */
+	@Deprecated
 	@Override
 	public Throwable getCause() {
 		return cause;

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/contexts/ContextManagerEvent.java

@@ -90,6 +90,7 @@ public final class ContextManagerEvent {
 	 *                                    enabledContextIdsChanged is
 	 *                                    <code>true</code>.
 	 */
+	@Deprecated
 	public ContextManagerEvent(IContextManager contextManager, boolean definedContextIdsChanged,
 			boolean enabledContextIdsChanged, Set previouslyDefinedContextIds, Set previouslyEnabledContextIds) {
 		if (contextManager == null) {
@@ -127,6 +128,7 @@ public ContextManagerEvent(IContextManager contextManager, boolean definedContex
 	 * @return the instance of the interface that changed. Guaranteed not to be
 	 *         <code>null</code>.
 	 */
+	@Deprecated
 	public IContextManager getContextManager() {
 		return contextManager;
 	}
@@ -141,6 +143,7 @@ public IContextManager getContextManager() {
 	 *         <code>false</code> and is guaranteed to not be null if
 	 *         haveDefinedContextIdsChanged() is <code>true</code>.
 	 */
+	@Deprecated
 	public Set getPreviouslyDefinedContextIds() {
 		return previouslyDefinedContextIds;
 	}
@@ -155,6 +158,7 @@ public Set getPreviouslyDefinedContextIds() {
 	 *         <code>false</code> and is guaranteed to not be null if
 	 *         haveEnabledContextIdsChanged() is <code>true</code>.
 	 */
+	@Deprecated
 	public Set getPreviouslyEnabledContextIds() {
 		return previouslyEnabledContextIds;
 	}
@@ -164,6 +168,7 @@ public Set getPreviouslyEnabledContextIds() {
 	 *
 	 * @return true, iff the definedContextIds property changed.
 	 */
+	@Deprecated
 	public boolean haveDefinedContextIdsChanged() {
 		return definedContextIdsChanged;
 	}
@@ -173,6 +178,7 @@ public boolean haveDefinedContextIdsChanged() {
 	 *
 	 * @return true, iff the enabledContextIds property changed.
 	 */
+	@Deprecated
 	public boolean haveEnabledContextIdsChanged() {
 		return enabledContextIdsChanged;
 	}

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/contexts/EnabledSubmission.java

@@ -91,6 +91,7 @@ public final class EnabledSubmission implements Comparable {
 	 * @param contextId               the identifier of the context to be enabled.
 	 *                                Must not be <code>null</code>.
 	 */
+	@Deprecated
 	public EnabledSubmission(String activePartId, Shell activeShell, IWorkbenchPartSite activeWorkbenchPartSite,
 			String contextId) {
 		if (contextId == null) {
@@ -106,6 +107,7 @@ public EnabledSubmission(String activePartId, Shell activeShell, IWorkbenchPartS
 	/**
 	 * @see Comparable#compareTo(java.lang.Object)
 	 */
+	@Deprecated
 	@Override
 	public int compareTo(Object object) {
 		EnabledSubmission castedObject = (EnabledSubmission) object;
@@ -133,6 +135,7 @@ public int compareTo(Object object) {
 	 * @return the identifier of the part that must be active for this request to be
 	 *         considered. May be <code>null</code>.
 	 */
+	@Deprecated
 	public String getActivePartId() {
 		return activePartId;
 	}
@@ -143,6 +146,7 @@ public String getActivePartId() {
 	 * @return the shell that must be active for this request to be considered. May
 	 *         be <code>null</code>.
 	 */
+	@Deprecated
 	public Shell getActiveShell() {
 		return activeShell;
 	}
@@ -154,6 +158,7 @@ public Shell getActiveShell() {
 	 * @return the workbench part site of the part that must be active for this
 	 *         request to be considered. May be <code>null</code>.
 	 */
+	@Deprecated
 	public IWorkbenchPartSite getActiveWorkbenchPartSite() {
 		return activeWorkbenchPartSite;
 	}
@@ -164,13 +169,15 @@ public IWorkbenchPartSite getActiveWorkbenchPartSite() {
 	 * @return the identifier of the context to be enabled. Guaranteed not to be
 	 *         <code>null</code>.
 	 */
+	@Deprecated
 	public String getContextId() {
 		return contextId;
 	}
 
 	/**
 	 * @see Object#toString()
 	 */
+	@Deprecated
 	@Override
 	public String toString() {
 		if (string == null) {