From 3ec383dfcab8ddc91f7fdcc10070158775aa0668 Mon Sep 17 00:00:00 2001
From: Natalia Ukhorskaya <natalia.ukhorskaya@jetbrains.com>
Date: Mon, 30 Nov 2015 10:41:11 +0300
Subject: [PATCH] Debugger: find context element in file copy using UserData

---
 .../jetbrains/kotlin/psi/KtCodeFragment.kt    | 11 +++++--
 .../extractFunctionForDebuggerUtil.kt         | 29 +++++++++----------
 2 files changed, 22 insertions(+), 18 deletions(-)

diff --git a/compiler/frontend/src/org/jetbrains/kotlin/psi/KtCodeFragment.kt b/compiler/frontend/src/org/jetbrains/kotlin/psi/KtCodeFragment.kt
index e77e1354de7..6de8bd95597 100644
--- a/compiler/frontend/src/org/jetbrains/kotlin/psi/KtCodeFragment.kt
+++ b/compiler/frontend/src/org/jetbrains/kotlin/psi/KtCodeFragment.kt
@@ -132,11 +132,16 @@ public abstract class KtCodeFragment(
     }
 
     public fun getContextContainingFile(): KtFile? {
-        val contextFile = (getContext() as? KtElement)?.getContainingKtFile()
+        return (getOriginalContext() as? KtElement)?.getContainingKtFile()
+    }
+
+    public fun getOriginalContext(): KtElement? {
+        val contextElement = getContext() as? KtElement
+        val contextFile = contextElement?.getContainingKtFile()
         if (contextFile is KtCodeFragment) {
-            return contextFile.getContextContainingFile()
+            return contextFile.getOriginalContext()
         }
-        return contextFile
+        return contextElement
     }
 
     private fun initImports(imports: String?) {
diff --git a/idea/src/org/jetbrains/kotlin/idea/debugger/evaluate/extractFunctionForDebuggerUtil.kt b/idea/src/org/jetbrains/kotlin/idea/debugger/evaluate/extractFunctionForDebuggerUtil.kt
index a0dd580a163..72be7358116 100644
--- a/idea/src/org/jetbrains/kotlin/idea/debugger/evaluate/extractFunctionForDebuggerUtil.kt
+++ b/idea/src/org/jetbrains/kotlin/idea/debugger/evaluate/extractFunctionForDebuggerUtil.kt
@@ -17,6 +17,7 @@
 package org.jetbrains.kotlin.idea.debugger.evaluate
 
 import com.intellij.debugger.engine.evaluation.EvaluateExceptionUtil
+import com.intellij.openapi.util.Key
 import com.intellij.psi.PsiElement
 import com.intellij.psi.PsiFile
 import com.intellij.psi.PsiManager
@@ -24,7 +25,6 @@ import com.intellij.psi.impl.PsiModificationTrackerImpl
 import org.jetbrains.kotlin.idea.actions.internal.KotlinInternalMode
 import org.jetbrains.kotlin.idea.caches.resolve.analyze
 import org.jetbrains.kotlin.idea.codeInsight.CodeInsightUtils
-import org.jetbrains.kotlin.idea.core.refactoring.createTempCopy
 import org.jetbrains.kotlin.idea.core.replaced
 import org.jetbrains.kotlin.idea.intentions.InsertExplicitTypeArgumentsIntention
 import org.jetbrains.kotlin.idea.refactoring.introduce.extractionEngine.*
@@ -34,8 +34,7 @@ import org.jetbrains.kotlin.idea.util.application.runReadAction
 import org.jetbrains.kotlin.idea.util.psi.patternMatching.toRange
 import org.jetbrains.kotlin.psi.*
 import org.jetbrains.kotlin.psi.codeFragmentUtil.suppressDiagnosticsInDebugMode
-import org.jetbrains.kotlin.psi.psiUtil.endOffset
-import org.jetbrains.kotlin.psi.psiUtil.startOffset
+import org.jetbrains.kotlin.psi.psiUtil.findDescendantOfType
 
 fun getFunctionForExtractedFragment(
         codeFragment: KtCodeFragment,
@@ -109,16 +108,25 @@ fun getFunctionForExtractedFragment(
 }
 
 fun addDebugExpressionIntoTmpFileForExtractFunction(originalFile: KtFile, codeFragment: KtCodeFragment, line: Int): List<KtExpression> {
-    val tmpFile = originalFile.createTempCopy { it }
+    val context = codeFragment.getOriginalContext()
+    context?.IS_CONTEXT_ELEMENT = true
+    val tmpFile = originalFile.copy() as KtFile
     tmpFile.suppressDiagnosticsInDebugMode = true
+    context?.IS_CONTEXT_ELEMENT = false
 
-    val contextElement = getExpressionToAddDebugExpressionBefore(tmpFile, codeFragment.context, line) ?: return emptyList()
+    val contextElement = getExpressionToAddDebugExpressionBefore(tmpFile, context, line) ?: return emptyList()
 
     addImportsToFile(codeFragment.importsAsImportList(), tmpFile)
 
     return addDebugExpressionBeforeContextElement(codeFragment, contextElement)
 }
 
+private var PsiElement.IS_CONTEXT_ELEMENT: Boolean by NotNullableCopyableUserDataProperty(Key.create("IS_CONTEXT_ELEMENT"), false)
+
+private fun KtFile.findContextElement(): KtElement? {
+    return this.findDescendantOfType { it.IS_CONTEXT_ELEMENT == true }
+}
+
 private fun addImportsToFile(newImportList: KtImportList?, tmpFile: KtFile) {
     if (newImportList != null && newImportList.imports.isNotEmpty()) {
         val tmpFileImportList = tmpFile.importList
@@ -139,10 +147,6 @@ private fun addImportsToFile(newImportList: KtImportList?, tmpFile: KtFile) {
     }
 }
 
-private fun KtFile.getElementInCopy(e: PsiElement): PsiElement? {
-    return CodeInsightUtils.findElementOfClassAtRange(this, e.startOffset, e.endOffset, e.javaClass)
-}
-
 private fun getExpressionToAddDebugExpressionBefore(tmpFile: KtFile, contextElement: PsiElement?, line: Int): PsiElement? {
     if (contextElement == null) {
         val lineStart = CodeInsightUtils.getStartLineOffset(tmpFile, line) ?: return null
@@ -152,14 +156,9 @@ private fun getExpressionToAddDebugExpressionBefore(tmpFile: KtFile, contextElem
         return CodeInsightUtils.getTopmostElementAtOffset(elementAtOffset, lineStart) ?: elementAtOffset
     }
 
-    val containingFile = contextElement.containingFile
-    if (containingFile is KtCodeFragment) {
-        return getExpressionToAddDebugExpressionBefore(tmpFile, containingFile.context, line)
-    }
-
     fun shouldStop(el: PsiElement?, p: PsiElement?) = p is KtBlockExpression || el is KtDeclaration || el is KtFile
 
-    var elementAt = tmpFile.getElementInCopy(contextElement)
+    var elementAt = tmpFile.findContextElement()
 
     var parent = elementAt?.parent
     if (shouldStop(elementAt, parent)) {