From 89b260739cf2578b89a64f3564109e598a6ac1a3 Mon Sep 17 00:00:00 2001
From: John Cortell <john.cortell@freescale.com>
Date: Fri, 9 Jan 2009 21:33:39 +0000
Subject: [PATCH] Fixed bugzilla 260595. Disable the Show Full Paths action
 when not applicable

---
 debug/org.eclipse.cdt.debug.ui/plugin.xml     | 12 +++++
 .../ui/actions/ShowFullPathsAction.java       |  5 +-
 .../internal/ui/actions/ViewFilterAction.java | 54 +++++++++++++++++--
 3 files changed, 67 insertions(+), 4 deletions(-)

diff --git a/debug/org.eclipse.cdt.debug.ui/plugin.xml b/debug/org.eclipse.cdt.debug.ui/plugin.xml
index 33e1d82bee4..4d98862c063 100644
--- a/debug/org.eclipse.cdt.debug.ui/plugin.xml
+++ b/debug/org.eclipse.cdt.debug.ui/plugin.xml
@@ -910,6 +910,12 @@
                class="org.eclipse.cdt.debug.internal.ui.actions.ShowFullPathsAction"
                menubarPath="cDebugActions"
                id="org.eclipse.cdt.debug.internal.ui.actions.ShowFullPathsAction">
+            <enablement>
+               <pluginState
+                     value="activated"
+                     id="org.eclipse.cdt.debug.ui">
+               </pluginState>
+            </enablement>
          </action>
       </viewContribution>
       <viewContribution
@@ -928,6 +934,12 @@
                class="org.eclipse.cdt.debug.internal.ui.actions.ShowFullPathsAction"
                menubarPath="cDebugActions"
                id="org.eclipse.cdt.debug.internal.ui.actions.ShowFullPathsAction">
+            <enablement>
+               <pluginState
+                     value="activated"
+                     id="org.eclipse.cdt.debug.ui">
+               </pluginState>
+            </enablement>
          </action>
          <action
                class="org.eclipse.cdt.debug.internal.ui.actions.AddEventBreakpointActionDelegate"
diff --git a/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/actions/ShowFullPathsAction.java b/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/actions/ShowFullPathsAction.java
index 86cf43b33d9..3c5aeee1d8f 100644
--- a/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/actions/ShowFullPathsAction.java
+++ b/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/actions/ShowFullPathsAction.java
@@ -24,7 +24,10 @@ import org.eclipse.jface.viewers.Viewer;
 import org.eclipse.swt.custom.BusyIndicator;
 
 /**
- * An action delegate that toggles the state of its viewer to show/hide full paths.
+ * An action delegate that toggles the state of its viewer to show/hide full
+ * paths. Note that we are not a filtering action (thus we unconditionally
+ * return true in {@link #select(Viewer, Object, Object)}), but we extend
+ * ViewFilterAction to get some basic, useful action behavior.
  */
 public class ShowFullPathsAction extends ViewFilterAction {
 
diff --git a/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/actions/ViewFilterAction.java b/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/actions/ViewFilterAction.java
index 643b7134a85..ade0e35ac93 100644
--- a/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/actions/ViewFilterAction.java
+++ b/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/actions/ViewFilterAction.java
@@ -10,7 +10,13 @@
  *******************************************************************************/
 package org.eclipse.cdt.debug.internal.ui.actions;
 
+import org.eclipse.debug.core.IBreakpointManager;
+import org.eclipse.debug.core.ILaunchManager;
+import org.eclipse.debug.core.model.IBreakpoint;
+import org.eclipse.debug.core.model.IDebugTarget;
 import org.eclipse.debug.ui.IDebugView;
+import org.eclipse.cdt.debug.core.model.ICBreakpoint;
+import org.eclipse.cdt.debug.core.model.ICDebugElement;
 import org.eclipse.cdt.debug.ui.CDebugUIPlugin;
 import org.eclipse.jface.action.IAction;
 import org.eclipse.jface.preference.IPreferenceStore;
@@ -24,7 +30,8 @@ import org.eclipse.ui.IViewActionDelegate;
 import org.eclipse.ui.IViewPart;
 
 /**
- * A base class for the view filtering actions.
+ * A base class for the CDT filtering actions added to views. We disable the action if
+ * the view has no CDT content.
  */
 public abstract class ViewFilterAction extends ViewerFilter implements IViewActionDelegate, IActionDelegate2 {
 	
@@ -87,10 +94,51 @@ public abstract class ViewFilterAction extends ViewerFilter implements IViewActi
 		CDebugUIPlugin.getDefault().savePluginPreferences();
 	}
 
-	/* (non-Javadoc)
-	 * @see org.eclipse.ui.IActionDelegate#selectionChanged(org.eclipse.jface.action.IAction, org.eclipse.jface.viewers.ISelection)
+	/**
+	 * Disable the action if there is no CDT content in the view. There is no
+	 * practical generic way to test that so we have to use view specific tests.
+	 * Currently, we support the Debug and Breakpoints view. Support for other
+	 * views should be added as needed.
+	 * 
+	 * Note that because we do this test on a view selection change, there can
+	 * be some edge cases where we'll be enabled even though there is no CDT
+	 * content. Closing those gaps would not be easy, and thus not worth the
+	 * effort as no harm is done by an unintentional enablement.
+	 * 
+	 * @see org.eclipse.ui.IActionDelegate#selectionChanged(org.eclipse.jface.action.IAction,
+	 *      org.eclipse.jface.viewers.ISelection)
 	 */
 	public void selectionChanged(IAction action, ISelection selection) {
+		boolean enable = false;
+		Object input = getStructuredViewer().getInput();
+		
+		// Debug view
+		if (input instanceof ILaunchManager) {
+			ILaunchManager launchmgr = (ILaunchManager)input;
+			IDebugTarget[] debugTargets = launchmgr.getDebugTargets();
+			for (IDebugTarget debugTarget : debugTargets) {
+				if (debugTarget instanceof ICDebugElement) {
+					enable = true;
+					break;
+				}
+			}
+		}
+		// Breakpoints view
+		else if (input instanceof IBreakpointManager) {
+			IBreakpointManager bkptmgr = (IBreakpointManager)input;
+			IBreakpoint[] bkpts = bkptmgr.getBreakpoints();
+			for (IBreakpoint bkpt : bkpts) {
+				if (bkpt instanceof ICBreakpoint) {
+					enable = true;
+					break;
+				}
+			}
+		}
+		// unsupported view; action will always be enabled.
+		else {
+			enable = true;
+		}
+		fAction.setEnabled(enable);
 	}
 
 	protected IPreferenceStore getPreferenceStore() {