[debug] coroutine's stack frame locations pre-calculated on initialization

idea/jvm-debugger/jvm-debugger-coroutine/src/org/jetbrains/kotlin/idea/debugger/coroutine/command/CoroutineBuilder.kt

@@ -200,10 +200,8 @@ class CreationCoroutineStackFrameItem(
     frame: StackFrameProxyImpl,
     val stackTraceElement: StackTraceElement,
     stackFrame: XStackFrame,
-    val location: Location
+    location: Location
-) : CoroutineStackFrameItem(frame, stackFrame) {
+) : CoroutineStackFrameItem(frame, stackFrame, location) {
-    override fun location() = location
-
     fun emptyDescriptor() =
         EmptyStackFrameDescriptor(stackTraceElement, frame)
 }
@@ -213,9 +211,7 @@ class SuspendCoroutineStackFrameItem(
     val stackTraceElement: StackTraceElement,
     stackFrame: XStackFrame,
     val lastObservedFrameFieldRef: ObjectReference
-) : CoroutineStackFrameItem(frame, stackFrame) {
+) : CoroutineStackFrameItem(frame, stackFrame, frame.location()) {
-    override fun location() = frame.location()
-
     fun emptyDescriptor() =
         EmptyStackFrameDescriptor(stackTraceElement, frame)
 }
@@ -224,28 +220,19 @@ class AsyncCoroutineStackFrameItem(
     frame: StackFrameProxyImpl,
     val frameItem: StackFrameItem,
     stackFrame: XStackFrame
-) : CoroutineStackFrameItem(frame, stackFrame) {
+) : CoroutineStackFrameItem(frame, stackFrame, frame.location())
-    override fun location() : Location = frame.location()
-}
 
 class RunningCoroutineStackFrameItem(
     frame: StackFrameProxyImpl,
     stackFrame: XStackFrame
-) : CoroutineStackFrameItem(frame, stackFrame) {
+) : CoroutineStackFrameItem(frame, stackFrame, frame.location())
-    val location = frame.location() // it should be invoked in manager thread
 
-    override fun location() = location
+abstract class CoroutineStackFrameItem(val frame: StackFrameProxyImpl, val stackFrame: XStackFrame, val location: Location) {
-}
-
-abstract class CoroutineStackFrameItem(val frame: StackFrameProxyImpl, val stackFrame: XStackFrame) {
     val log by logger
 
     fun sourcePosition() : XSourcePosition? = stackFrame.sourcePosition
 
-    abstract fun location(): Location
-
     fun uniqueId(): String {
-        val location = location()
         try {
             return location.safeSourceName() + ":" + location.safeMethod().toString() + ":" +
                     location.safeLineNumber() + ":" + location.safeSourceLineNumber()

idea/jvm-debugger/jvm-debugger-coroutine/src/org/jetbrains/kotlin/idea/debugger/coroutine/view/XCoroutineView.kt

@@ -237,7 +237,7 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
     inner class CoroutineFrameValue(val frame: CoroutineStackFrameItem
     ) : XNamedValue(frame.uniqueId()) {
         override fun computePresentation(node: XValueNode, place: XValuePlace) =
-            applyRenderer(node, renderer.render(frame.location()))
+            applyRenderer(node, renderer.render(frame.location))
     }
 
     private fun applyRenderer(node: XValueNode, presentation: SimpleColoredTextIcon) =