Enhanced toggling

Enhanced toggling to support direct switch to different types of
breakpoints without multiple attempts

Author: subyssurendran666

org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/actions/RulerToggleHitBreakpointActionDelegate.java

@@ -13,6 +13,8 @@
  *******************************************************************************/
 package org.eclipse.jdt.internal.debug.ui.actions;
 
+import org.eclipse.core.runtime.CoreException;
+import org.eclipse.debug.internal.ui.DebugUIPlugin;
 import org.eclipse.jdt.debug.core.IJavaLineBreakpoint;
 import org.eclipse.jdt.internal.debug.ui.JDIDebugUIPlugin;
 import org.eclipse.jface.dialogs.IInputValidator;
@@ -27,14 +29,27 @@ public class RulerToggleHitBreakpointActionDelegate extends AbstractRulerToggleB
 	@Override
 	protected boolean doWork(ITextEditor editor, ITextSelection selection) {
 		IJavaLineBreakpoint jlp = ToggleBreakpointAdapter.findExistingBreakpoint(currentEditor, selection);
-		if (jlp == null) {
-			hitCountDialog();
-			if (BreakpointToggleUtils.getHitCount() < 1) {
-				return false;
+
+		try {
+			if (jlp != null && jlp.getHitCount() < 1) {
+				hitCountDialog();
+				if (BreakpointToggleUtils.getHitCount() < 1) {
+					return false;
+				}
+				ToggleBreakpointAdapter.deleteBreakpoint(jlp, editor, null);
+			}
+			if (jlp == null) {
+				hitCountDialog();
+				if (BreakpointToggleUtils.getHitCount() < 1) {
+					return false;
+				}
 			}
+			BreakpointToggleUtils.setHitpoint(true);
+			return true;
+		} catch (CoreException e) {
+			DebugUIPlugin.log(e);
+			return false;
 		}
-		BreakpointToggleUtils.setHitpoint(true);
-		return true;
 	}
 
 	private void hitCountDialog() {

org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/actions/RulerToggleTracepointActionDelegate.java

@@ -13,14 +13,28 @@
  *******************************************************************************/
 package org.eclipse.jdt.internal.debug.ui.actions;
 
+import org.eclipse.core.runtime.CoreException;
+import org.eclipse.debug.internal.ui.DebugUIPlugin;
+import org.eclipse.jdt.debug.core.IJavaLineBreakpoint;
 import org.eclipse.jface.text.ITextSelection;
 import org.eclipse.ui.texteditor.ITextEditor;
 
 public class RulerToggleTracepointActionDelegate extends AbstractRulerToggleBreakpointActionDelegate {
 
 	@Override
 	protected boolean doWork(ITextEditor editor, ITextSelection selection) {
-		BreakpointToggleUtils.setUnsetTracepoint(true);
-		return true;
+		IJavaLineBreakpoint jlp = ToggleBreakpointAdapter.findExistingBreakpoint(currentEditor, selection);
+		try {
+			if (jlp != null && !jlp.isConditionEnabled()) {
+				ToggleBreakpointAdapter.deleteBreakpoint(jlp, editor, null);
+				BreakpointToggleUtils.setUnsetTracepoint(true);
+				return true;
+			}
+			BreakpointToggleUtils.setUnsetTracepoint(true);
+			return true;
+		} catch (CoreException e) {
+			DebugUIPlugin.log(e);
+			return false;
+		}
 	}
-}
+}

org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/actions/RulerToggleTriggerBreakpointActionDelegate.java

@@ -13,14 +13,28 @@
  *******************************************************************************/
 package org.eclipse.jdt.internal.debug.ui.actions;
 
+import org.eclipse.core.runtime.CoreException;
+import org.eclipse.debug.internal.ui.DebugUIPlugin;
+import org.eclipse.jdt.debug.core.IJavaLineBreakpoint;
 import org.eclipse.jface.text.ITextSelection;
 import org.eclipse.ui.texteditor.ITextEditor;
 
 public class RulerToggleTriggerBreakpointActionDelegate extends AbstractRulerToggleBreakpointActionDelegate {
 
 	@Override
 	protected boolean doWork(ITextEditor editor, ITextSelection selection) {
-		BreakpointToggleUtils.setTriggerpoint(true);
-		return true;
+		IJavaLineBreakpoint jlp = ToggleBreakpointAdapter.findExistingBreakpoint(currentEditor, selection);
+		try {
+			if (jlp != null && !jlp.isTriggerPoint()) {
+				ToggleBreakpointAdapter.deleteBreakpoint(jlp, editor, null);
+				BreakpointToggleUtils.setTriggerpoint(true);
+				return true;
+			}
+			BreakpointToggleUtils.setTriggerpoint(true);
+			return true;
+		} catch (CoreException e) {
+			DebugUIPlugin.log(e);
+			return false;
+		}
 	}
 }

org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/actions/ToggleBreakpointAdapter.java

@@ -1683,7 +1683,7 @@ private static void logBadAnnotation(SimpleMarkerAnnotation annotation, CoreExce
 	 * @param progressMonitor the progress monitor
 	 * @throws CoreException if the deletion fails
 	 */
-	private static void deleteBreakpoint(IJavaBreakpoint breakpoint, IWorkbenchPart part, IProgressMonitor monitor) throws CoreException {
+	public static void deleteBreakpoint(IJavaBreakpoint breakpoint, IWorkbenchPart part, IProgressMonitor monitor) throws CoreException {
 		final Shell shell = part != null ? part.getSite().getShell() : null;
 		final boolean[] result = new boolean[] { true };
 
@@ -1878,4 +1878,4 @@ private static ICompilationUnit getCompilationUnit(ITextEditor editor) {
 		return JavaCore.createCompilationUnitFrom(file);
 	}
 
-}
+}