Avoid trying to get callstack[0] when it's empty like in an instrumentation pause (#82852)

thaystg · web-flow · commit b89c0dd63e93 · 2023-03-02T10:25:52.000-03:00
diff --git a/src/mono/wasm/debugger/BrowserDebugProxy/MonoProxy.cs b/src/mono/wasm/debugger/BrowserDebugProxy/MonoProxy.cs
@@ -183,6 +183,9 @@ protected override async Task<bool> AcceptEvent(SessionId sessionId, JObject par
 
         protected async Task<bool> OnDebuggerPaused(SessionId sessionId, JObject args, CancellationToken token)
         {
+            if (args?["callFrames"]?.Value<JArray>()?.Count == 0) //new browser version can send pause of type "instrumentation" with an empty callstack
+                return false;
+
             if (args["asyncStackTraceId"] != null)
             {
                 if (!contexts.TryGetValue(sessionId, out ExecutionContext context))

Original file line number	Diff line number	Diff line change
`@@ -183,6 +183,9 @@ protected override async Task<bool> AcceptEvent(SessionId sessionId, JObject par`
`183`	`183`
`184`	`184`	`protected async Task<bool> OnDebuggerPaused(SessionId sessionId, JObject args, CancellationToken token)`
`185`	`185`	`{`
	`186`	`+ if (args?["callFrames"]?.Value<JArray>()?.Count == 0) //new browser version can send pause of type "instrumentation" with an empty callstack`
	`187`	`+ return false;`
	`188`	`+`
`186`	`189`	`if (args["asyncStackTraceId"] != null)`
`187`	`190`	`{`
`188`	`191`	`if (!contexts.TryGetValue(sessionId, out ExecutionContext context))`