Skip to content

Commit

Permalink
Implementing support to Debugger::Break. (#44305)
Browse files Browse the repository at this point in the history
  • Loading branch information
thaystg committed Nov 5, 2020
1 parent 3d9bef9 commit 29ec808
Show file tree
Hide file tree
Showing 4 changed files with 33 additions and 1 deletion.
7 changes: 7 additions & 0 deletions src/mono/mono/mini/mini-wasm-debugger.c
Original file line number Diff line number Diff line change
Expand Up @@ -444,6 +444,7 @@ mono_wasm_debugger_init (void)
objrefs = g_hash_table_new_full (NULL, NULL, NULL, mono_debugger_free_objref);

mini_get_dbg_callbacks ()->handle_exception = handle_exception;
mini_get_dbg_callbacks ()->user_break = mono_wasm_user_break;
}

MONO_API void
Expand Down Expand Up @@ -663,6 +664,12 @@ mono_wasm_breakpoint_hit (void)
// mono_wasm_fire_bp ();
}

void
mono_wasm_user_break (void)
{
mono_wasm_fire_bp ();
}

EMSCRIPTEN_KEEPALIVE int
mono_wasm_current_bp_id (void)
{
Expand Down
1 change: 1 addition & 0 deletions src/mono/mono/mini/mini-wasm.h
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,7 @@ void mono_wasm_set_timeout (int timeout, int id);

void mono_wasm_single_step_hit (void);
void mono_wasm_breakpoint_hit (void);
void mono_wasm_user_break (void);

int mono_wasm_assembly_already_added (const char *assembly_name);

Expand Down
17 changes: 17 additions & 0 deletions src/mono/wasm/debugger/DebuggerTestSuite/Tests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1158,6 +1158,23 @@ async Task LoadAssemblyDynamically(string asm_file, string pdb_file)
Assert.True(load_assemblies_res.IsOk);
}

[Fact]
public async Task BreakOnDebuggerBreak()
{
var insp = new Inspector();
//Collect events
var scripts = SubscribeToScripts(insp);

await Ready();
await insp.Ready(async (cli, token) =>
{
ctx = new DebugTestContext(cli, insp, token, scripts);
await EvaluateAndCheck(
"window.setTimeout(function() { invoke_static_method_async('[debugger-test] UserBreak:BreakOnDebuggerBreakCommand'); }, 1);",
"dotnet://debugger-test.dll/debugger-test2.cs", 56, 4,
"BreakOnDebuggerBreakCommand");
});
}
//TODO add tests covering basic stepping behavior as step in/out/over
}
}
9 changes: 8 additions & 1 deletion src/mono/wasm/debugger/tests/debugger-test/debugger-test2.cs
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
// The .NET Foundation licenses this file to you under the MIT license.

using System;

using System.Diagnostics;
public class Misc
{ //Only append content to this class as the test suite depends on line info
public static int CreateObject(int foo, int bar)
Expand Down Expand Up @@ -49,3 +49,10 @@ public static void Types()
var d = usMin + usMax;
}
}

public class UserBreak {
public static void BreakOnDebuggerBreakCommand()
{
Debugger.Break();
}
}

0 comments on commit 29ec808

Please sign in to comment.