Debugger: remove unnecessary whitespaces in presentation of line breakpoint variants

Author: NataliaUkhorskaya

ChangeLog.md

@@ -15,6 +15,7 @@
 
 #### Debugger
 
+- Fix text with line breaks in popup with line breakpoint variants
 - Do not cast to runtime type unavailable in current scope
 
 ## 1.0.2

idea/src/org/jetbrains/kotlin/idea/debugger/breakpoints/KotlinLineBreakpointType.java

@@ -30,6 +30,7 @@
 import com.intellij.xdebugger.XSourcePosition;
 import com.intellij.xdebugger.breakpoints.XBreakpoint;
 import com.intellij.xdebugger.breakpoints.XLineBreakpoint;
+import kotlin.text.StringsKt;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 import org.jetbrains.java.debugger.breakpoints.properties.JavaBreakpointProperties;
@@ -155,6 +156,11 @@ public KotlinLineBreakpointVariant(XSourcePosition position, PsiElement element)
             super(position, element, -1);
         }
 
+        @Override
+        public String getText() {
+            return StringsKt.replace(super.getText(), "  ", "", true);
+        }
+
         @Override
         public Icon getIcon() {
             return KotlinIcons.FUNCTION;

idea/src/org/jetbrains/kotlin/idea/debugger/breakpoints/breakpointTypeUtils.kt

@@ -36,7 +36,9 @@ import org.jetbrains.kotlin.idea.refactoring.getLineNumber
 import org.jetbrains.kotlin.idea.refactoring.getLineStartOffset
 import org.jetbrains.kotlin.psi.*
 import org.jetbrains.kotlin.psi.psiUtil.endOffset
+import org.jetbrains.kotlin.psi.psiUtil.parentsWithSelf
 import org.jetbrains.kotlin.psi.psiUtil.startOffset
+import org.jetbrains.kotlin.utils.addToStdlib.firstIsInstance
 import java.util.*
 
 fun canPutAt(file: VirtualFile, line: Int, project: Project, breakpointTypeClass: Class<*>): Boolean {
@@ -96,11 +98,12 @@ fun computeVariants(
 
     val result = LinkedList<JavaLineBreakpointType.JavaBreakpointVariant>()
 
-    val mainMethod = KotlinLineBreakpointType.getContainingMethod(pos.elementAt)
+    val elementAt = pos.elementAt.parentsWithSelf.firstIsInstance<KtElement>()
+    val mainMethod = KotlinLineBreakpointType.getContainingMethod(elementAt)
     if (mainMethod != null) {
         result.add(kotlinBreakpointType.KotlinLineBreakpointVariant(
                 XSourcePositionImpl.createByElement(mainMethod),
-                CodeInsightUtils.getTopmostElementAtOffset(pos.elementAt, pos.offset) ?: mainMethod))
+                CodeInsightUtils.getTopmostElementAtOffset(elementAt, pos.offset) ?: mainMethod))
     }
 
     lambdas.forEachIndexed { ordinal, lambda ->