Attachment 181458 Details for Bug 328422 – patch

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

patch

key_binding_service.patch (text/plain), 12.21 KB, created by Elias Volanakis

on 2010-10-21 20:40:40 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Elias Volanakis

Created: 2010-10-21 20:40:40 EDT

Size: 12.21 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.rap.ui.workbench
>Index: Eclipse UI/org/eclipse/ui/IKeyBindingService.java
>===================================================================
>RCS file: /cvsroot/rt/org.eclipse.rap/runtime.ui/org.eclipse.rap.ui.workbench/Eclipse UI/org/eclipse/ui/IKeyBindingService.java,v
>retrieving revision 1.2
>diff -u -r1.2 IKeyBindingService.java
>--- Eclipse UI/org/eclipse/ui/IKeyBindingService.java	21 Apr 2008 14:55:42 -0000	1.2
>+++ Eclipse UI/org/eclipse/ui/IKeyBindingService.java	22 Oct 2010 00:28:18 -0000
>@@ -1,68 +1,67 @@
>-// RAP [rh] Deprecated interface IKeyBindingService not implemented
>-///*******************************************************************************
>-// * Copyright (c) 2000, 2006 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.ui;
>-//
>-//import org.eclipse.jface.action.IAction;
>-//
>-///**
>-// * The key binding service allows one to query or set the scope of Eclipse for
>-// * the purposes of resolving key assignments to commands, and to register
>-// * actions to handle specific commands. See the <code>org.eclipse.ui.commands</code>
>-// * extension point for details.
>-// * 
>-// * A participating workbench part is responsible to register all its actions
>-// * with this service. The part is also responsible to set the current scope.
>-// * 
>-// * 
>-// * This interface is not intended to be implemented or extended by clients.
>-// * 
>-// * 
>-// * @since 2.0
>-// * @deprecated See IContextService to manage scopes and
>-// * IHandlerService to manage handlers. IAction can
>-// * be proxied by org.eclipse.jface.commands.ActionHandler.
>-// * @noimplement This interface is not intended to be implemented by clients.
>-// */
>-//public interface IKeyBindingService {
>-//
>-// /**
>-// * Returns the active accelerator scope ids.
>-// * 
>-// * @return the active accelerator scope ids.
>-// */
>-// String[] getScopes();
>-//
>-// /**
>-// * Registers an action with the key binding service.
>-// * 
>-// * @param action
>-// * the action to be registered with the key binding service.
>-// */
>-// void registerAction(IAction action);
>-//
>-// /**
>-// * Sets the active accelerator scope ids.
>-// * 
>-// * @param scopes
>-// * the active accelerator scope ids.
>-// */
>-// void setScopes(String[] scopes);
>-//
>-// /**
>-// * Unregisters an action with the key binding service.
>-// * 
>-// * @param action
>-// * the action to be unregistered with the key binding service.
>-// */
>-// void unregisterAction(IAction action);
>-//}
>+/*******************************************************************************
>+ * Copyright (c) 2000, 2006 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.ui;
>+
>+import org.eclipse.jface.action.IAction;
>+
>+/**
>+ * The key binding service allows one to query or set the scope of Eclipse for
>+ * the purposes of resolving key assignments to commands, and to register
>+ * actions to handle specific commands. See the <code>org.eclipse.ui.commands</code>
>+ * extension point for details.
>+ * 
>+ * A participating workbench part is responsible to register all its actions
>+ * with this service. The part is also responsible to set the current scope.
>+ * 
>+ * 
>+ * This interface is not intended to be implemented or extended by clients.
>+ * 
>+ * 
>+ * @since 1.4
>+ * @deprecated See IContextService to manage scopes and
>+ * IHandlerService to manage handlers. IAction can
>+ * be proxied by org.eclipse.jface.commands.ActionHandler.
>+ * @noimplement This interface is not intended to be implemented by clients.
>+ */
>+public interface IKeyBindingService {
>+
>+ /**
>+ * Returns the active accelerator scope ids.
>+ * 
>+ * @return the active accelerator scope ids.
>+ */
>+ String[] getScopes();
>+
>+ /**
>+ * Registers an action with the key binding service.
>+ * 
>+ * @param action
>+ * the action to be registered with the key binding service.
>+ */
>+ void registerAction(IAction action);
>+
>+ /**
>+ * Sets the active accelerator scope ids.
>+ * 
>+ * @param scopes
>+ * the active accelerator scope ids.
>+ */
>+ void setScopes(String[] scopes);
>+
>+ /**
>+ * Unregisters an action with the key binding service.
>+ * 
>+ * @param action
>+ * the action to be unregistered with the key binding service.
>+ */
>+ void unregisterAction(IAction action);
>+}
>Index: Eclipse UI/org/eclipse/ui/INestableKeyBindingService.java
>===================================================================
>RCS file: /cvsroot/rt/org.eclipse.rap/runtime.ui/org.eclipse.rap.ui.workbench/Eclipse UI/org/eclipse/ui/INestableKeyBindingService.java,v
>retrieving revision 1.2
>diff -u -r1.2 INestableKeyBindingService.java
>--- Eclipse UI/org/eclipse/ui/INestableKeyBindingService.java	21 Apr 2008 14:55:42 -0000	1.2
>+++ Eclipse UI/org/eclipse/ui/INestableKeyBindingService.java	22 Oct 2010 00:28:18 -0000
>@@ -1,72 +1,71 @@
>-// RAP [rh] Deprecated interface INestableKeyBindingService not implemented
>-///*******************************************************************************
>-// * Copyright (c) 2000, 2006 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.ui;
>-//
>-//import org.eclipse.ui.services.IServiceLocator;
>-//
>-///**
>-// * 
>-// * A service that is capable of nesting other services within itself. This 
>-// * allows lower level components to query for a service provider in a 
>-// * hierarchical fashion, and for information to be resolved in a hierarchical
>-// * manner
>-// * 
>-// * 
>-// * This interface is not intended to be implemented or extended by clients.
>-// * 
>-// * 
>-// * @since 2.1.3
>-// * @deprecated This is now handled by {@link IServiceLocator} which can
>-// * be nested.
>-// * @noimplement This interface is not intended to be implemented by clients.
>-// */
>-//public interface INestableKeyBindingService extends IKeyBindingService {
>-//
>-// /**
>-// * Marks the service associated with <code>nestedSite</code> as active if
>-// * one exists. If there is no service associated, then nothing changes.
>-// * Calling this method with <code>null</code> forces deactivation of the
>-// * current service.
>-// * 
>-// * @param nestedSite The site whose service should be activated; 
>-// * <code>null</code> if the current service should be deactivated.
>-// * @return <code>true</code> if a service is activated (or deactivated, in
>-// * the case of a <code>null</code> parameter); <code>false</code> if
>-// * nothing changed.
>-// */
>-// public boolean activateKeyBindingService(IWorkbenchSite nestedSite);
>-//
>-// /**
>-// * An accessor for the nested key binding service associated with a
>-// * particular site. If the key binding service does not exist for this
>-// * <code>nestedSite</code> already, then a new one should be constructed.
>-// * 
>-// * @param nestedSite The site for which the service should be found; 
>-// * should not be <code>null</code>.
>-// * @return The associated service, if any; or a new associated service, if
>-// * none existed previously.
>-// */
>-// public IKeyBindingService getKeyBindingService(IWorkbenchSite nestedSite);
>-//
>-// /**
>-// * Removes a nested key binding service from this key binding service. The
>-// * service to remove is determined by the <code>nestedSite</code> with
>-// * which it is associated.
>-// * 
>-// * @param nestedSite The site from which to remove the nested service.
>-// * This site must not be <code>null</code>.
>-// * @return <code>true</code> if the service existed and could be removed; 
>-// * <code>false</code> otherwise.
>-// */
>-// public boolean removeKeyBindingService(IWorkbenchSite nestedSite);
>-//
>-//}
>+/*******************************************************************************
>+ * Copyright (c) 2000, 2006 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.ui;
>+
>+import org.eclipse.ui.services.IServiceLocator;
>+
>+/**
>+ * 
>+ * A service that is capable of nesting other services within itself. This 
>+ * allows lower level components to query for a service provider in a 
>+ * hierarchical fashion, and for information to be resolved in a hierarchical
>+ * manner
>+ * 
>+ * 
>+ * This interface is not intended to be implemented or extended by clients.
>+ * 
>+ * 
>+ * @since 1.4
>+ * @deprecated This is now handled by {@link IServiceLocator} which can
>+ * be nested.
>+ * @noimplement This interface is not intended to be implemented by clients.
>+ */
>+public interface INestableKeyBindingService extends IKeyBindingService {
>+
>+ /**
>+ * Marks the service associated with <code>nestedSite</code> as active if
>+ * one exists. If there is no service associated, then nothing changes.
>+ * Calling this method with <code>null</code> forces deactivation of the
>+ * current service.
>+ * 
>+ * @param nestedSite The site whose service should be activated; 
>+ * <code>null</code> if the current service should be deactivated.
>+ * @return <code>true</code> if a service is activated (or deactivated, in
>+ * the case of a <code>null</code> parameter); <code>false</code> if
>+ * nothing changed.
>+ */
>+ public boolean activateKeyBindingService(IWorkbenchSite nestedSite);
>+
>+ /**
>+ * An accessor for the nested key binding service associated with a
>+ * particular site. If the key binding service does not exist for this
>+ * <code>nestedSite</code> already, then a new one should be constructed.
>+ * 
>+ * @param nestedSite The site for which the service should be found; 
>+ * should not be <code>null</code>.
>+ * @return The associated service, if any; or a new associated service, if
>+ * none existed previously.
>+ */
>+ public IKeyBindingService getKeyBindingService(IWorkbenchSite nestedSite);
>+
>+ /**
>+ * Removes a nested key binding service from this key binding service. The
>+ * service to remove is determined by the <code>nestedSite</code> with
>+ * which it is associated.
>+ * 
>+ * @param nestedSite The site from which to remove the nested service.
>+ * This site must not be <code>null</code>.
>+ * @return <code>true</code> if the service existed and could be removed; 
>+ * <code>false</code> otherwise.
>+ */
>+ public boolean removeKeyBindingService(IWorkbenchSite nestedSite);
>+
>+}

Actions: View

Attachments on bug 328422: 181458