some refactor. on review

This commit is contained in:
andrey.zaytsev
2013-03-18 19:15:59 +04:00
parent 783608595b
commit a0e2851584
2 changed files with 4 additions and 10 deletions
@@ -187,7 +187,7 @@ public class BreakpointsFavoriteListProvider extends AbstractFavoritesListProvid
case EDIT:
DebuggerSupport debuggerSupport = XBreakpointUtil.getDebuggerSupport(myProject, breakpointItem);
if (debuggerSupport == null) return;
debuggerSupport.getEditBreakpointAction().editBreakpoint(myProject, component, whereToShow, breakpointItem.getBreakpoint());
debuggerSupport.getEditBreakpointAction().editBreakpoint(myProject, component, whereToShow, breakpointItem);
break;
case REMOVE:
breakpointItem.removed(myProject);
@@ -25,19 +25,13 @@ import com.intellij.openapi.editor.markup.GutterIconRenderer;
import com.intellij.openapi.project.Project;
import com.intellij.openapi.util.Pair;
import com.intellij.xdebugger.impl.breakpoints.XBreakpointUtil;
import com.intellij.xdebugger.impl.breakpoints.ui.BreakpointItem;
import com.intellij.xdebugger.impl.breakpoints.ui.BreakpointsMasterDetailPopupFactory;
import org.jetbrains.annotations.NotNull;
import javax.swing.*;
import java.awt.*;
/**
* Created with IntelliJ IDEA.
* User: zajac
* Date: 04.05.12
* Time: 4:03
* To change this template use File | Settings | File Templates.
*/
public abstract class EditBreakpointActionHandler extends DebuggerActionHandler {
protected abstract void doShowPopup(Project project, JComponent component, Point whereToShow, Object breakpoint);
@@ -67,7 +61,7 @@ public abstract class EditBreakpointActionHandler extends DebuggerActionHandler
doShowPopup(project, gutterComponent, whereToShow, breakpoint);
}
public void editBreakpoint(@NotNull Project project, @NotNull JComponent parent, @NotNull Point whereToShow, @NotNull Object breakpoint) {
doShowPopup(project, parent, whereToShow, breakpoint);
public void editBreakpoint(@NotNull Project project, @NotNull JComponent parent, @NotNull Point whereToShow, @NotNull BreakpointItem breakpoint) {
doShowPopup(project, parent, whereToShow, breakpoint.getBreakpoint());
}
}