Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPawel Piech2009-05-06 15:14:48 -0400
committerPawel Piech2009-05-06 15:14:48 -0400
commit5cf2394fc6fb47fa9715c442f240e6f9d6909de7 (patch)
treecbd84b408f655bafef95cff7752f3c9c7854250a
parentc80532136688c9ca02b853e856287162378c5b49 (diff)
downloadeclipse.platform.debug-5cf2394fc6fb47fa9715c442f240e6f9d6909de7.tar.gz
eclipse.platform.debug-5cf2394fc6fb47fa9715c442f240e6f9d6909de7.tar.xz
eclipse.platform.debug-5cf2394fc6fb47fa9715c442f240e6f9d6909de7.zip
Bug 275009 - Remove the "Next/Previous Thread" feature in 3.5
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/IDebugHelpContextIds.java2
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/NavigateModelAction.java194
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/provisional/IModelNavigateProxy.java26
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/provisional/IModelNavigateUpdate.java38
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/DebugTargetProxy.java37
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/LaunchView.java10
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/NextThreadNavAction.java41
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/PreviousThreadNavAction.java41
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractDebugView.java2
9 files changed, 4 insertions, 387 deletions
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/IDebugHelpContextIds.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/IDebugHelpContextIds.java
index 3dd3bfd6e..ace7df94b 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/IDebugHelpContextIds.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/IDebugHelpContextIds.java
@@ -74,8 +74,6 @@ public interface IDebugHelpContextIds {
public static final String DEBUG_VIEW_MODE_AUTO_ACTION = PREFIX + "debug_view_mode_auto_action_context"; //$NON-NLS-1$
public static final String DEBUG_VIEW_MODE_FULL_ACTION = PREFIX + "debug_view_mode_full_action_context"; //$NON-NLS-1$
public static final String DEBUG_VIEW_MODE_COMPACT_ACTION = PREFIX + "debug_view_mode_compact_action_context"; //$NON-NLS-1$
- public static final String NEXT_THREAD_NAVIGATE_ACTION = PREFIX + "next_thread_navigate_action_context"; //$NON-NLS-1$
- public static final String PREVIOUS_THREAD_NAVIGATE_ACTION = PREFIX + "previous_thread_navigate_action_context"; //$NON-NLS-1$
public static final String DEBUG_VIEW_DROP_DOWN_AUTOEXPAND_ACTION = PREFIX + "debug_view_drop_down_autoexpand_action_context"; //$NON-NLS-1$
// Views
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/NavigateModelAction.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/NavigateModelAction.java
deleted file mode 100644
index 765f5f3f8..000000000
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/NavigateModelAction.java
+++ /dev/null
@@ -1,194 +0,0 @@
-/*******************************************************************************
-
- * Copyright (c) 2009 Wind River Systems 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:
- * Wind River Systems - initial API and implementation
- *******************************************************************************/
-package org.eclipse.debug.internal.ui.viewers.model;
-
-import java.util.Arrays;
-
-import org.eclipse.debug.internal.ui.viewers.model.provisional.IModelDelta;
-import org.eclipse.debug.internal.ui.viewers.model.provisional.IModelNavigateProxy;
-import org.eclipse.debug.internal.ui.viewers.model.provisional.IModelNavigateUpdate;
-import org.eclipse.debug.internal.ui.viewers.model.provisional.IModelProxy;
-import org.eclipse.debug.internal.ui.viewers.model.provisional.TreeModelViewer;
-import org.eclipse.jface.action.Action;
-import org.eclipse.jface.viewers.ISelection;
-import org.eclipse.jface.viewers.ISelectionChangedListener;
-import org.eclipse.jface.viewers.ITreeSelection;
-import org.eclipse.jface.viewers.SelectionChangedEvent;
-import org.eclipse.jface.viewers.TreePath;
-
-/**
- * Action used to navigate the model in the treemodel viewer. This action uses
- * the {@link IModelNavigateProxy} to select the next or previous element in the
- * model. The model determines which elements should be navigated to (e.g. next
- * suspended thread).
- *
- * @since 3.5
- */
-public class NavigateModelAction extends Action implements ISelectionChangedListener {
-
- final private TreeModelViewer fViewer;
- final private boolean fReverse;
-
- public NavigateModelAction(TreeModelViewer viewer, boolean reverse) {
- fViewer = viewer;
- fReverse = reverse;
- fViewer.addSelectionChangedListener(this);
- update();
- }
-
- public void dispose() {
- fViewer.removeSelectionChangedListener(this);
- }
-
- public void selectionChanged(SelectionChangedEvent event) {
- update();
- }
-
- private void update() {
- // Update the action enablement. Action should be disabled if there
- // are no model proxies that support navigation.
- boolean enabled = false;
- ModelContentProvider contentProvider = (ModelContentProvider)fViewer.getContentProvider();
- final IModelProxy[] proxies = contentProvider.getModelProxies();
- for (int i = 0; i < proxies.length; i++) {
- if (proxies[i] instanceof IModelNavigateProxy) {
- enabled = true;
- }
- }
- setEnabled(enabled);
- }
-
-
- /**
- * Update class for collecting the model delta to navigate the viewer to.
- */
- abstract private class ModelNavigateUpdate extends ViewerUpdateMonitor implements IModelNavigateUpdate {
-
- final IModelNavigateProxy fModelTraversalProxy;
- IModelDelta fDelta;
-
- ModelNavigateUpdate(IModelNavigateProxy proxy, TreePath path) {
- super((ModelContentProvider)fViewer.getContentProvider(),
- fViewer.getInput(),
- path,
- path.getSegmentCount() == 0 ? fViewer.getInput() : path.getLastSegment(),
- null,
- fViewer.getPresentationContext());
- fModelTraversalProxy = proxy;
- }
-
- public void setNextElementDelta(IModelDelta delta) {
- fDelta = delta;
- }
-
- public boolean isReverse() {
- return fReverse;
- }
-
- void startRequest() {
- fModelTraversalProxy.update(this);
- }
-
- boolean coalesce(ViewerUpdateMonitor update) {
- return false;
- }
-
- int getPriority() {
- return 0;
- }
-
- TreePath getSchedulingPath() {
- return getElementPath();
- }
- }
-
- public void run() {
- // Get the model proxy of the currently selected element.
- setEnabled(false);
- TreePath path = null;
- final ISelection currentContext = fViewer.getSelection();
- if (currentContext instanceof ITreeSelection) {
- TreePath[] paths = ((ITreeSelection)currentContext).getPaths();
- if (paths.length > 0 && paths[0].getSegmentCount() > 0) {
- path = paths[0];
- }
- }
- if (path == null) {
- path = TreePath.EMPTY;
- }
- ModelContentProvider contentProvider = (ModelContentProvider)fViewer.getContentProvider();
- IModelProxy selectionProxy = contentProvider.getElementProxy(path);
-
- // Calculate the index of the current proxy.
- final IModelProxy[] proxies = contentProvider.getModelProxies();
- int _idx = Arrays.asList(proxies).indexOf(selectionProxy);
- final int selectionProxyIdx = _idx == -1 ? 0 : _idx;
- final int nextProxyIdx = getNextProxyIdx(proxies, selectionProxyIdx);
-
- // Try traversing in the current model.
- if (selectionProxy instanceof IModelNavigateProxy && !selectionProxy.isDisposed()) {
- new ModelNavigateUpdate((IModelNavigateProxy)selectionProxy, path) {
- protected void performUpdate() {
- if (fDelta != null) {
- fViewer.updateViewer(fDelta);
- setEnabled(true);
- } else if (proxies.length > 0) {
- updateNextProxy(proxies, nextProxyIdx, nextProxyIdx, true);
- } else {
- setEnabled(true);
- }
- }
- }.start();
- } else if (proxies.length > 0) {
- updateNextProxy(proxies, nextProxyIdx, nextProxyIdx, true);
- }
- }
-
- private int getNextProxyIdx(final IModelProxy[] proxies, int currentProxyIdx) {
- int _nextProxyIdx = currentProxyIdx;
- if (fReverse) {
- _nextProxyIdx--;
- if (_nextProxyIdx < 0) {
- _nextProxyIdx = proxies.length - 1;
- }
- } else {
- _nextProxyIdx++;
- if (_nextProxyIdx >= proxies.length) {
- _nextProxyIdx = 0;
- }
- }
- return _nextProxyIdx;
- }
-
- private void updateNextProxy(final IModelProxy[] proxies, final int currentProxyIdx, final int endProxyIdx, boolean start) {
- IModelProxy proxy = proxies[currentProxyIdx];
-
- if (!start && currentProxyIdx == endProxyIdx) {
- // We're finished trying to find the next context
- setEnabled(true);
- } else if (proxy instanceof IModelNavigateProxy && !proxy.isDisposed()) {
- new ModelNavigateUpdate((IModelNavigateProxy)proxy, TreePath.EMPTY) {
- protected void performUpdate() {
- if (fDelta != null) {
- fViewer.updateViewer(fDelta);
- setEnabled(true);
- } else {
- updateNextProxy(proxies, getNextProxyIdx(proxies, currentProxyIdx), endProxyIdx, false);
- }
- }
- }.start();
- } else {
- updateNextProxy(proxies, getNextProxyIdx(proxies, currentProxyIdx), endProxyIdx, false);
- }
-
- }
-}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/provisional/IModelNavigateProxy.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/provisional/IModelNavigateProxy.java
deleted file mode 100644
index d3c03a0dc..000000000
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/provisional/IModelNavigateProxy.java
+++ /dev/null
@@ -1,26 +0,0 @@
-/*******************************************************************************
- * Copyright (c) 2009 Wind River Systems 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:
- * Wind River Systems - initial API and implementation
- *******************************************************************************/
-package org.eclipse.debug.internal.ui.viewers.model.provisional;
-
-/**
- * Model proxy extension that allows the viewer to navigate elements in the model.
- *
- * @since 3.5
- */
-public interface IModelNavigateProxy extends IModelProxy {
-
- /**
- * Update to retrieve the next element in the model to navigate to.
- *
- * @param update Update to complete.
- */
- public void update(IModelNavigateUpdate update);
-}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/provisional/IModelNavigateUpdate.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/provisional/IModelNavigateUpdate.java
deleted file mode 100644
index 606ca5b6b..000000000
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/model/provisional/IModelNavigateUpdate.java
+++ /dev/null
@@ -1,38 +0,0 @@
-/*******************************************************************************
- * Copyright (c) 2009 Wind River Systems 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:
- * Wind River Systems - initial API and implementation
- *******************************************************************************/
-package org.eclipse.debug.internal.ui.viewers.model.provisional;
-
-
-/**
- * Update to retrieve the next element to navigate to in model.
- *
- * @since 3.5
- */
-public interface IModelNavigateUpdate extends IViewerUpdate {
-
- /**
- * True if navigation is in reverse.
- * @return
- */
- public boolean isReverse();
-
- /**
- * Sets the delta which will select the next element to navigate to. The delta
- * should use the {@link IModelDelta.FORCE} flag in conjunction with the
- * {@link IModelDelta.SELECT} flag. If the update element is the last element
- * in the navigation order, this delta should be set to <code>null</code>
- *
- * @param delta Delta which will cause next model element to be revealed. Should
- * be set to <code>null</code> to indicate that there are no more elements to
- * navigate to in the given traversal order.
- */
- public void setNextElementDelta(IModelDelta delta);
-}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/DebugTargetProxy.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/DebugTargetProxy.java
index 915337234..b8ba83927 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/DebugTargetProxy.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/DebugTargetProxy.java
@@ -10,10 +10,6 @@
*******************************************************************************/
package org.eclipse.debug.internal.ui.viewers.update;
-import org.eclipse.core.runtime.IProgressMonitor;
-import org.eclipse.core.runtime.IStatus;
-import org.eclipse.core.runtime.Status;
-import org.eclipse.core.runtime.jobs.Job;
import org.eclipse.debug.core.DebugEvent;
import org.eclipse.debug.core.DebugException;
import org.eclipse.debug.core.DebugPlugin;
@@ -25,8 +21,6 @@ import org.eclipse.debug.core.model.IDebugTarget;
import org.eclipse.debug.core.model.IStackFrame;
import org.eclipse.debug.core.model.IThread;
import org.eclipse.debug.internal.ui.viewers.model.provisional.IModelDelta;
-import org.eclipse.debug.internal.ui.viewers.model.provisional.IModelNavigateProxy;
-import org.eclipse.debug.internal.ui.viewers.model.provisional.IModelNavigateUpdate;
import org.eclipse.debug.internal.ui.viewers.model.provisional.ModelDelta;
import org.eclipse.jface.viewers.Viewer;
@@ -35,7 +29,7 @@ import org.eclipse.jface.viewers.Viewer;
*
* @since 3.2
*/
-public class DebugTargetProxy extends EventHandlerModelProxy implements IModelNavigateProxy {
+public class DebugTargetProxy extends EventHandlerModelProxy {
private IDebugTarget fDebugTarget;
@@ -82,7 +76,7 @@ public class DebugTargetProxy extends EventHandlerModelProxy implements IModelNa
// select any thread that is already suspended after installation
IDebugTarget target = fDebugTarget;
if (target != null) {
- ModelDelta delta = getNextSuspendedThreadDelta(null, false, false);
+ ModelDelta delta = getNextSuspendedThreadDelta(null, false);
if (delta == null) {
ILaunchManager manager = DebugPlugin.getDefault().getLaunchManager();
ILaunch launch = target.getLaunch();
@@ -97,30 +91,7 @@ public class DebugTargetProxy extends EventHandlerModelProxy implements IModelNa
}
}
- public void update(final IModelNavigateUpdate update) {
- new Job("Traverse Debug Model") { //$NON-NLS-1$
- { setSystem(true); }
- protected IStatus run(IProgressMonitor monitor) {
- // Get current thread based on selection
- IThread currentThread = null;
- Object element = update.getElement();
- if (element instanceof IThread) {
- currentThread = (IThread)element;
- }
- else if (element instanceof IStackFrame) {
- currentThread = ((IStackFrame)element).getThread();
- }
-
- // Calculate next thread and complete update.
- update.setNextElementDelta( getNextSuspendedThreadDelta(currentThread, update.isReverse(), true) );
- update.done();
-
- return Status.OK_STATUS;
- }
- }.schedule();
- }
-
- protected ModelDelta getNextSuspendedThreadDelta(IThread currentThread, boolean reverse, boolean force) {
+ protected ModelDelta getNextSuspendedThreadDelta(IThread currentThread, boolean reverse) {
IDebugTarget target = fDebugTarget;
if (target != null) {
try {
@@ -161,7 +132,7 @@ public class DebugTargetProxy extends EventHandlerModelProxy implements IModelNa
ModelDelta node = delta.addNode(launch, launchIndex, IModelDelta.NO_CHANGE, target.getLaunch().getChildren().length);
node = node.addNode(target, targetIndex, IModelDelta.NO_CHANGE, threads.length);
node = node.addNode(chosen, threadIndex, IModelDelta.NO_CHANGE | IModelDelta.EXPAND, chosen.getStackFrames().length);
- node = node.addNode(frame, 0, IModelDelta.NO_CHANGE | IModelDelta.SELECT | (force ? IModelDelta.FORCE : 0), 0);
+ node = node.addNode(frame, 0, IModelDelta.NO_CHANGE | IModelDelta.SELECT, 0);
return delta;
}
}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/LaunchView.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/LaunchView.java
index edc7f1b37..3131d4954 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/LaunchView.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/LaunchView.java
@@ -116,7 +116,6 @@ import org.eclipse.ui.IWorkbenchWindow;
import org.eclipse.ui.PartInitException;
import org.eclipse.ui.WorkbenchException;
import org.eclipse.ui.XMLMemento;
-import org.eclipse.ui.actions.ActionFactory;
import org.eclipse.ui.actions.SelectionListenerAction;
import org.eclipse.ui.dialogs.PropertyDialogAction;
import org.eclipse.ui.part.IPageBookViewPage;
@@ -492,8 +491,6 @@ public class LaunchView extends AbstractDebugView implements ISelectionChangedLi
fEditSourceAction = new EditSourceLookupPathAction(this);
fLookupAction = new LookupSourceAction(this);
setAction(FIND_ACTION, new VirtualFindAction((InternalTreeModelViewer) getViewer()));
- setAction(ActionFactory.NEXT.getId(), new NextThreadNavAction((TreeModelViewer)getViewer()));
- setAction(ActionFactory.PREVIOUS.getId(), new PreviousThreadNavAction((TreeModelViewer)getViewer()));
addCapabilityAction(new TerminateCommandAction(), TERMINATE);
addCapabilityAction(new DisconnectCommandAction(), DISCONNECT);
@@ -599,8 +596,6 @@ public class LaunchView extends AbstractDebugView implements ISelectionChangedLi
// default page site.
IActionBars bars = getViewSite().getActionBars();
pageActionBars.setGlobalActionHandler(FIND_ACTION, bars.getGlobalActionHandler(FIND_ACTION));
- pageActionBars.setGlobalActionHandler(ActionFactory.NEXT.getId(), bars.getGlobalActionHandler(ActionFactory.NEXT.getId()));
- pageActionBars.setGlobalActionHandler(ActionFactory.PREVIOUS.getId(), bars.getGlobalActionHandler(ActionFactory.PREVIOUS.getId()));
}
/**
@@ -1024,9 +1019,6 @@ public class LaunchView extends AbstractDebugView implements ISelectionChangedLi
disposeCommandAction(TERMINATE_AND_REMOVE);
disposeCommandAction(TERMINATE_AND_RELAUNCH);
disposeCommandAction(TERMINATE_ALL);
-
- ((NextThreadNavAction)getAction(ActionFactory.NEXT.getId())).dispose();
- ((PreviousThreadNavAction)getAction(ActionFactory.PREVIOUS.getId())).dispose();
}
/**
@@ -1114,8 +1106,6 @@ public class LaunchView extends AbstractDebugView implements ISelectionChangedLi
menu.add(new Separator(IDebugUIConstants.EMPTY_EDIT_GROUP));
menu.add(new Separator(IDebugUIConstants.EDIT_GROUP));
menu.add(getAction(FIND_ACTION));
- menu.add(getAction(ActionFactory.NEXT.getId()));
- menu.add(getAction(ActionFactory.PREVIOUS.getId()));
menu.add(new Separator(IDebugUIConstants.EMPTY_STEP_GROUP));
menu.add(new Separator(IDebugUIConstants.STEP_GROUP));
menu.add(new GroupMarker(IDebugUIConstants.STEP_INTO_GROUP));
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/NextThreadNavAction.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/NextThreadNavAction.java
deleted file mode 100644
index 83a8a935c..000000000
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/NextThreadNavAction.java
+++ /dev/null
@@ -1,41 +0,0 @@
-/*******************************************************************************
- * Copyright (c) 2009 Wind River Systems 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:
- * Wind River Systems - initial API and implementation
- * IBM Corporation - updated to use command constants
- *******************************************************************************/
-package org.eclipse.debug.internal.ui.views.launch;
-
-import org.eclipse.ui.IWorkbenchCommandConstants;
-import org.eclipse.ui.PlatformUI;
-
-import org.eclipse.debug.internal.ui.DebugUIPlugin;
-import org.eclipse.debug.internal.ui.IDebugHelpContextIds;
-import org.eclipse.debug.internal.ui.IInternalDebugUIConstants;
-import org.eclipse.debug.internal.ui.actions.ActionMessages;
-import org.eclipse.debug.internal.ui.viewers.model.NavigateModelAction;
-import org.eclipse.debug.internal.ui.viewers.model.provisional.TreeModelViewer;
-
-import org.eclipse.debug.ui.DebugUITools;
-
-/**
- * Navigates to the next suspended thread in the Debug view.
- *
- * @since 3.5
- */
-class NextThreadNavAction extends NavigateModelAction {
- public NextThreadNavAction(TreeModelViewer viewer) {
- super(viewer, false);
- setText(ActionMessages.NextThreadNavAction_name);
- setImageDescriptor(DebugUITools.getImageDescriptor(IInternalDebugUIConstants.IMG_ELCL_NEXT_THREAD));
- setDisabledImageDescriptor(DebugUITools.getImageDescriptor(IInternalDebugUIConstants.IMG_ELCL_NEXT_THREAD));
- setId(DebugUIPlugin.getUniqueIdentifier() + ".NextThreadNavigateAction"); //$NON-NLS-1$
- PlatformUI.getWorkbench().getHelpSystem().setHelp(this, IDebugHelpContextIds.NEXT_THREAD_NAVIGATE_ACTION);
- setActionDefinitionId(IWorkbenchCommandConstants.NAVIGATE_NEXT);
- }
-}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/PreviousThreadNavAction.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/PreviousThreadNavAction.java
deleted file mode 100644
index 42a3fd953..000000000
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/launch/PreviousThreadNavAction.java
+++ /dev/null
@@ -1,41 +0,0 @@
-/*******************************************************************************
- * Copyright (c) 2009 Wind River Systems 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:
- * Wind River Systems - initial API and implementation
- * IBM Corporation - updated to use command constants
- *******************************************************************************/
-package org.eclipse.debug.internal.ui.views.launch;
-
-import org.eclipse.ui.IWorkbenchCommandConstants;
-import org.eclipse.ui.PlatformUI;
-
-import org.eclipse.debug.internal.ui.DebugUIPlugin;
-import org.eclipse.debug.internal.ui.IDebugHelpContextIds;
-import org.eclipse.debug.internal.ui.IInternalDebugUIConstants;
-import org.eclipse.debug.internal.ui.actions.ActionMessages;
-import org.eclipse.debug.internal.ui.viewers.model.NavigateModelAction;
-import org.eclipse.debug.internal.ui.viewers.model.provisional.TreeModelViewer;
-
-import org.eclipse.debug.ui.DebugUITools;
-
-/**
- * Navigates to the previous suspended thread in the Debug view.
- *
- * @since 3.5
- */
-class PreviousThreadNavAction extends NavigateModelAction {
- public PreviousThreadNavAction(TreeModelViewer viewer) {
- super(viewer, true);
- setText(ActionMessages.PreviousThreadNavAction_name);
- setImageDescriptor(DebugUITools.getImageDescriptor(IInternalDebugUIConstants.IMG_ELCL_PREVIOUS_THREAD));
- setDisabledImageDescriptor(DebugUITools.getImageDescriptor(IInternalDebugUIConstants.IMG_ELCL_PREVIOUS_THREAD));
- setId(DebugUIPlugin.getUniqueIdentifier() + ".PreviousThreadNavigateAction"); //$NON-NLS-1$
- PlatformUI.getWorkbench().getHelpSystem().setHelp(this, IDebugHelpContextIds.PREVIOUS_THREAD_NAVIGATE_ACTION);
- setActionDefinitionId(IWorkbenchCommandConstants.NAVIGATE_PREVIOUS);
- }
-}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractDebugView.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractDebugView.java
index 2f0278bab..a0eb8f5b5 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractDebugView.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractDebugView.java
@@ -160,8 +160,6 @@ public abstract class AbstractDebugView extends PageBookView implements IDebugVi
fgGlobalActionIds.add(FIND_ACTION);
fgGlobalActionIds.add(ActionFactory.UNDO.getId());
fgGlobalActionIds.add(ActionFactory.REDO.getId());
- fgGlobalActionIds.add(ActionFactory.NEXT.getId());
- fgGlobalActionIds.add(ActionFactory.PREVIOUS.getId());
}
/**

Back to the top