Wasm: fix issues in debugger

core/src/main/java/org/teavm/debugging/Debugger.java

@@ -162,6 +162,17 @@ public class Debugger {
         return Promise.allVoid(jsBreakpointPromises);
     }
 
+    private Promise<Void> destroyTemporaryBreakpoints() {
+        var temporaryBreakpoints = new ArrayList<>(this.temporaryBreakpoints);
+        this.temporaryBreakpoints.clear();
+        var promises = new ArrayList<Promise<Void>>();
+        for (var jsBreakpoint : temporaryBreakpoints) {
+            promises.add(jsBreakpoint.destroy());
+        }
+        callStack = null;
+        return Promise.allVoid(promises);
+    }
+
     private boolean addJsBreakpoints(CallFrame frame, JavaScriptScript script, boolean enterMethod,
             Set<JavaScriptLocation> successors) {
         var debugInfo = debugInformationMap.get(script);
@@ -197,10 +208,10 @@ public class Debugger {
         var callAddresses = IntHashSet.from(wasmStepLocationsFinder.getCallAddresses());
         var result = createTemporaryBreakpoints(locations, br -> {
             if (br != null && br.isValid() && callAddresses.contains(br.getLocation().getColumn())) {
-                javaScriptDebugger.stepInto();
+                destroyTemporaryBreakpoints().thenVoid(x -> javaScriptDebugger.stepInto());
-                return false;
-            }
                 return true;
+            }
+            return false;
         });
         return result.thenVoid(x -> javaScriptDebugger.stepOut());
     }
@@ -659,16 +670,9 @@ public class Debugger {
     }
 
     private void firePaused(JavaScriptBreakpoint breakpoint) {
-        var temporaryBreakpoints = new ArrayList<>(this.temporaryBreakpoints);
         var handler = temporaryBreakpointHandler;
-        this.temporaryBreakpoints.clear();
         temporaryBreakpointHandler = null;
-        var promises = new ArrayList<Promise<Void>>();
-        for (var jsBreakpoint : temporaryBreakpoints) {
-            promises.add(jsBreakpoint.destroy());
-        }
         callStack = null;
-        Promise.allVoid(promises).thenVoid(v -> {
         Breakpoint javaBreakpoint = null;
         JavaScriptBreakpoint tmpBreakpoint = null;
         if (breakpoint != null) {
@@ -679,12 +683,14 @@ public class Debugger {
             }
         }
         if (handler == null || !handler.test(tmpBreakpoint)) {
+            var pausedAtBreakpoint = javaBreakpoint;
+            destroyTemporaryBreakpoints().thenVoid(v -> {
                 for (var listener : getListeners()) {
-                    listener.paused(javaBreakpoint);
+                    listener.paused(pausedAtBreakpoint);
-                }
                 }
             });
         }
+    }
 
     private void fireAttached() {
         for (Breakpoint breakpoint : breakpoints) {

tools/chrome-rdp/src/main/java/org/teavm/chromerdp/ChromeRDPDebugger.java

@@ -484,9 +484,10 @@ public class ChromeRDPDebugger extends BaseChromeRDPDebugger implements JavaScri
                     return new RDPValue(this, null, remoteValue.getType(), remoteValue.getObjectId(),
                             true);
                 }
-            default:
+            default: {
-                return new RDPValue(this, remoteValue.getValue().asText(), remoteValue.getType(),
+                var valueAsText = remoteValue.getValue() != null ? remoteValue.getValue().asText() : "null";
-                        remoteValue.getObjectId(), false);
+                return new RDPValue(this, valueAsText, remoteValue.getType(), remoteValue.getObjectId(), false);
+            }
         }
     }