From f1b0f113bfda8edd023c4a09a68f87e6ea084bc6 Mon Sep 17 00:00:00 2001 From: Sergey Simonchik Date: Fri, 6 Sep 2013 19:03:14 +0400 Subject: [PATCH] xdebug api improvement --- .../src/com/intellij/xdebugger/XDebugProcess.java | 9 ++------- .../intellij/xdebugger/ui/XDebugLayoutCustomizer.java | 3 ++- .../com/intellij/xdebugger/impl/ui/XDebugSessionTab.java | 2 +- 3 files changed, 5 insertions(+), 9 deletions(-) diff --git a/platform/xdebugger-api/src/com/intellij/xdebugger/XDebugProcess.java b/platform/xdebugger-api/src/com/intellij/xdebugger/XDebugProcess.java index 1c3deec9c080..ad0622b476f6 100644 --- a/platform/xdebugger-api/src/com/intellij/xdebugger/XDebugProcess.java +++ b/platform/xdebugger-api/src/com/intellij/xdebugger/XDebugProcess.java @@ -44,7 +44,6 @@ import javax.swing.event.HyperlinkListener; public abstract class XDebugProcess { private final XDebugSession mySession; private ProcessHandler myProcessHandler; - private XDebugLayoutCustomizer myLayoutCustomizer; /** * @param session pass session parameter of {@link XDebugProcessStarter#start} method to this constructor @@ -198,12 +197,8 @@ public abstract class XDebugProcess { } @Nullable - public XDebugLayoutCustomizer getLayoutCustomizer() { - return myLayoutCustomizer; - } - - public void setLayoutCustomizer(@Nullable XDebugLayoutCustomizer layoutCustomizer) { - myLayoutCustomizer = layoutCustomizer; + public XDebugLayoutCustomizer createLayoutCustomizer() { + return null; } /** diff --git a/platform/xdebugger-api/src/com/intellij/xdebugger/ui/XDebugLayoutCustomizer.java b/platform/xdebugger-api/src/com/intellij/xdebugger/ui/XDebugLayoutCustomizer.java index fa02615a08e4..f5a9ec39e00a 100644 --- a/platform/xdebugger-api/src/com/intellij/xdebugger/ui/XDebugLayoutCustomizer.java +++ b/platform/xdebugger-api/src/com/intellij/xdebugger/ui/XDebugLayoutCustomizer.java @@ -11,6 +11,7 @@ import org.jetbrains.annotations.NotNull; * @author Sergey Simonchik */ public interface XDebugLayoutCustomizer { + /** * Registers tab for the given {@code console}, that is returned by {@link com.intellij.xdebugger.XDebugProcess#createConsole()}. * @@ -26,4 +27,4 @@ public interface XDebugLayoutCustomizer { * @param ui {@code RunnerLayoutUi} instance */ void registerAdditionalContent(@NotNull RunnerLayoutUi ui); -} \ No newline at end of file +} diff --git a/platform/xdebugger-impl/src/com/intellij/xdebugger/impl/ui/XDebugSessionTab.java b/platform/xdebugger-impl/src/com/intellij/xdebugger/impl/ui/XDebugSessionTab.java index b407be66570f..7b77f4fa096a 100644 --- a/platform/xdebugger-impl/src/com/intellij/xdebugger/impl/ui/XDebugSessionTab.java +++ b/platform/xdebugger-impl/src/com/intellij/xdebugger/impl/ui/XDebugSessionTab.java @@ -136,7 +136,7 @@ public class XDebugSessionTab extends DebuggerSessionTabBase { myUi.addContent(createFramesContent(session), 0, PlaceInGrid.left, false); myUi.addContent(createVariablesContent(session), 0, PlaceInGrid.center, false); myUi.addContent(createWatchesContent(session, sessionData), 0, PlaceInGrid.right, false); - XDebugLayoutCustomizer layoutCustomizer = debugProcess.getLayoutCustomizer(); + XDebugLayoutCustomizer layoutCustomizer = debugProcess.createLayoutCustomizer(); final Content consoleContent; if (layoutCustomizer != null) { consoleContent = layoutCustomizer.registerConsoleContent(myConsole, myUi);