[COROUTINE] Information message added if breakpoint policy doesn't suspended all

threads.
This commit is contained in:
Vladimir Ilmov
2020-01-30 18:44:36 +01:00
parent eff02b6e72
commit 8d4dac398d
2 changed files with 46 additions and 28 deletions
@@ -50,7 +50,7 @@ class CoroutineViewDebugSessionListener(
} }
fun renew(suspendContext: XSuspendContext) { fun renew(suspendContext: XSuspendContext) {
if(suspendContext is SuspendContextImpl && suspendContext.suspendPolicy == EventRequest.SUSPEND_ALL) { if (suspendContext is SuspendContextImpl) {
DebuggerUIUtil.invokeLater { DebuggerUIUtil.invokeLater {
xCoroutineView.renewRoot(suspendContext) xCoroutineView.renewRoot(suspendContext)
} }
@@ -12,7 +12,6 @@ import com.intellij.debugger.engine.SuspendContextImpl
import com.intellij.debugger.engine.evaluation.EvaluationContextImpl import com.intellij.debugger.engine.evaluation.EvaluationContextImpl
import com.intellij.debugger.jdi.StackFrameProxyImpl import com.intellij.debugger.jdi.StackFrameProxyImpl
import com.intellij.debugger.jdi.ThreadReferenceProxyImpl import com.intellij.debugger.jdi.ThreadReferenceProxyImpl
import com.intellij.debugger.memory.utils.StackFrameItem
import com.intellij.ide.CommonActionsManager import com.intellij.ide.CommonActionsManager
import com.intellij.openapi.Disposable import com.intellij.openapi.Disposable
import com.intellij.openapi.actionSystem.ActionManager import com.intellij.openapi.actionSystem.ActionManager
@@ -27,7 +26,6 @@ import com.intellij.ui.CaptionPanel
import com.intellij.ui.ComboboxSpeedSearch import com.intellij.ui.ComboboxSpeedSearch
import com.intellij.ui.DoubleClickListener import com.intellij.ui.DoubleClickListener
import com.intellij.ui.border.CustomLineBorder import com.intellij.ui.border.CustomLineBorder
import com.intellij.ui.components.JBLabel
import com.intellij.ui.components.panels.Wrapper import com.intellij.ui.components.panels.Wrapper
import com.intellij.util.SingleAlarm import com.intellij.util.SingleAlarm
import com.intellij.xdebugger.XDebugSession import com.intellij.xdebugger.XDebugSession
@@ -42,12 +40,11 @@ import com.intellij.xdebugger.impl.ui.tree.XDebuggerTreeRestorer
import com.intellij.xdebugger.impl.ui.tree.XDebuggerTreeState import com.intellij.xdebugger.impl.ui.tree.XDebuggerTreeState
import com.intellij.xdebugger.impl.ui.tree.nodes.XValueContainerNode import com.intellij.xdebugger.impl.ui.tree.nodes.XValueContainerNode
import com.intellij.xdebugger.impl.ui.tree.nodes.XValueNodeImpl import com.intellij.xdebugger.impl.ui.tree.nodes.XValueNodeImpl
import javaslang.control.Either import com.sun.jdi.request.EventRequest
import org.jetbrains.kotlin.idea.KotlinBundle import org.jetbrains.kotlin.idea.KotlinBundle
import org.jetbrains.kotlin.idea.debugger.coroutine.CoroutineDebuggerContentInfo import org.jetbrains.kotlin.idea.debugger.coroutine.CoroutineDebuggerContentInfo
import org.jetbrains.kotlin.idea.debugger.coroutine.CoroutineDebuggerContentInfo.Companion.XCOROUTINE_POPUP_ACTION_GROUP import org.jetbrains.kotlin.idea.debugger.coroutine.CoroutineDebuggerContentInfo.Companion.XCOROUTINE_POPUP_ACTION_GROUP
import org.jetbrains.kotlin.idea.debugger.coroutine.VersionedImplementationProvider import org.jetbrains.kotlin.idea.debugger.coroutine.VersionedImplementationProvider
import org.jetbrains.kotlin.idea.debugger.coroutine.command.*
import org.jetbrains.kotlin.idea.debugger.coroutine.data.* import org.jetbrains.kotlin.idea.debugger.coroutine.data.*
import org.jetbrains.kotlin.idea.debugger.coroutine.proxy.* import org.jetbrains.kotlin.idea.debugger.coroutine.proxy.*
import org.jetbrains.kotlin.idea.debugger.coroutine.util.CreateContentParams import org.jetbrains.kotlin.idea.debugger.coroutine.util.CreateContentParams
@@ -163,8 +160,15 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
inner class CoroutineGroupContainer(val suspendContext: XSuspendContext, val groupName: String) : XValueContainer() { inner class CoroutineGroupContainer(val suspendContext: XSuspendContext, val groupName: String) : XValueContainer() {
override fun computeChildren(node: XCompositeNode) { override fun computeChildren(node: XCompositeNode) {
if (suspendContext is SuspendContextImpl && suspendContext.suspendPolicy == EventRequest.SUSPEND_ALL) {
val groups = XValueChildrenList.singleton(CoroutineContainer(suspendContext, groupName)) val groups = XValueChildrenList.singleton(CoroutineContainer(suspendContext, groupName))
node.addChildren(groups, true) node.addChildren(groups, true)
} else {
node.addChildren(
XValueChildrenList.singleton(ErrorNode("To enable information breakpoint suspend policy should be set to 'All' threads.")),
true,
)
}
} }
} }
@@ -231,7 +235,8 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
} }
} }
inner class CoroutineFrameValue(val frame: CoroutineStackFrameItem inner class CoroutineFrameValue(
val frame: CoroutineStackFrameItem
) : XNamedValue(frame.uniqueId()) { ) : XNamedValue(frame.uniqueId()) {
override fun computePresentation(node: XValueNode, place: XValuePlace) = override fun computePresentation(node: XValueNode, place: XValuePlace) =
applyRenderer(node, renderer.render(frame.location)) applyRenderer(node, renderer.render(frame.location))
@@ -262,13 +267,15 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
nodeSelected(KeyMouseEvent(e)) nodeSelected(KeyMouseEvent(e))
}.installOn(tree) }.installOn(tree)
tree.addKeyListener(object : KeyAdapter() { tree.addKeyListener(
object : KeyAdapter() {
override fun keyPressed(e: KeyEvent) { override fun keyPressed(e: KeyEvent) {
val key = e.keyCode val key = e.keyCode
if (key == KeyEvent.VK_ENTER || key == KeyEvent.VK_SPACE || key == KeyEvent.VK_RIGHT) if (key == KeyEvent.VK_ENTER || key == KeyEvent.VK_SPACE || key == KeyEvent.VK_RIGHT)
nodeSelected(KeyMouseEvent(e)) nodeSelected(KeyMouseEvent(e))
} }
}) },
)
} }
fun nodeSelected(event: KeyMouseEvent): Boolean { fun nodeSelected(event: KeyMouseEvent): Boolean {
@@ -286,7 +293,8 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
createStackAndSetFrame(threadProxy, { frame.stackFrame }, isCurrentContext) createStackAndSetFrame(threadProxy, { frame.stackFrame }, isCurrentContext)
} }
is CreationCoroutineStackFrameItem -> { is CreationCoroutineStackFrameItem -> {
val position = getPosition(frame.stackTraceElement.className, frame.stackTraceElement.lineNumber) ?: return false val position =
getPosition(frame.stackTraceElement.className, frame.stackTraceElement.lineNumber) ?: return false
val threadProxy = threadSuspendContext.thread as ThreadReferenceProxyImpl val threadProxy = threadSuspendContext.thread as ThreadReferenceProxyImpl
createStackAndSetFrame(threadProxy, { SyntheticStackFrame(frame.emptyDescriptor(), emptyList(), position) }) createStackAndSetFrame(threadProxy, { SyntheticStackFrame(frame.emptyDescriptor(), emptyList(), position) })
} }
@@ -298,7 +306,10 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
is RestoredCoroutineStackFrameItem -> { is RestoredCoroutineStackFrameItem -> {
val threadProxy = frame.frame.threadProxy() val threadProxy = frame.frame.threadProxy()
val position = getPosition(frame.location.declaringType().name(), frame.location.lineNumber()) ?: return false val position = getPosition(frame.location.declaringType().name(), frame.location.lineNumber()) ?: return false
createStackAndSetFrame(threadProxy, { SyntheticStackFrame(frame.emptyDescriptor(), frame.spilledVariables, position) }) createStackAndSetFrame(
threadProxy,
{ SyntheticStackFrame(frame.emptyDescriptor(), frame.spilledVariables, position) }
)
} }
else -> { else -> {
@@ -310,7 +321,11 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
} }
} }
fun createStackAndSetFrame(threadReferenceProxy: ThreadReferenceProxyImpl, stackFrameProvider: () -> XStackFrame?, isCurrentContext: Boolean = false) { fun createStackAndSetFrame(
threadReferenceProxy: ThreadReferenceProxyImpl,
stackFrameProvider: () -> XStackFrame?,
isCurrentContext: Boolean = false
) {
val threadSuspendContext = session.suspendContext as SuspendContextImpl val threadSuspendContext = session.suspendContext as SuspendContextImpl
managerThreadExecutor.on(threadSuspendContext).schedule { managerThreadExecutor.on(threadSuspendContext).schedule {
val stackFrame = stackFrameProvider.invoke() val stackFrame = stackFrameProvider.invoke()
@@ -319,7 +334,8 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
applicationThreadExecutor.schedule( applicationThreadExecutor.schedule(
{ {
session.setCurrentStackFrame(executionStack, stackFrame) session.setCurrentStackFrame(executionStack, stackFrame)
}, panel.tree },
panel.tree
) )
} }
} }
@@ -357,7 +373,9 @@ class XCoroutineView(val project: Project, val session: XDebugSession) :
frame: SuspendCoroutineStackFrameItem frame: SuspendCoroutineStackFrameItem
): SyntheticStackFrame? { ): SyntheticStackFrame? {
val position = applicationThreadExecutor.readAction { getPosition(frame.stackTraceElement.className, frame.stackTraceElement.lineNumber) } ?: return null val position =
applicationThreadExecutor.readAction { getPosition(frame.stackTraceElement.className, frame.stackTraceElement.lineNumber) }
?: return null
val lookupContinuation = LookupContinuation(executionContext, frame.stackTraceElement) val lookupContinuation = LookupContinuation(executionContext, frame.stackTraceElement)
val continuation = lookupContinuation.findContinuation(frame.lastObservedFrameFieldRef) ?: return null val continuation = lookupContinuation.findContinuation(frame.lastObservedFrameFieldRef) ?: return null