[mono][debugger] Fix Debugger.Break() behavior when running on release mode

[thaystg]

If native_debugger_break is disabled do not call G_BREAKPOINT otherwise the app will be interrupted by a signal when running in release mode a code that has a Debugger.Break()

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1675269

[ghost]

Tagging subscribers to this area: @thaystg
See info in area-owners.md if you want to be subscribed.

Issue Details

---

If native_debugger_break is disabled do not call G_BREAKPOINT otherwise the app will be interrupted by a signal when running in release mode a code that has a Debugger.Break()

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1675269

Author:	thaystg
Assignees:	-
Labels:	area-Debugger-mono
Milestone:	-

[thaystg]

@steveisok should we backport this one?

[thaystg]

@lewing should we backport this one?

[steveisok]

Yes, I think so as long as @lewing is ok w/ it.

[thaystg]

/backport to release/7.0

[github-actions]

Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/3893630409

[lambdageek]

@thaystg Should we backport this to 6.0, too?

runtime/src/mono/mono/component/debugger-stub.c

Lines 156 to 159 in ace6275

	stub_debugger_user_break (void)
	{
	G_BREAKPOINT ();
	}

[thaystg]

@thaystg Should we backport this to 6.0, too?

runtime/src/mono/mono/component/debugger-stub.c

Lines 156 to 159 in ace6275

	stub_debugger_user_break (void)
	{
	G_BREAKPOINT ();
	}

I think so. I will open it. :)