[wasm][debugger] Removed internalProperties group.

[ilonatommy]

Fixes #75778.

Motivation: #75778 (comment).

It fixes a bug from net 7, so will need backporting.

[ghost]

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Fixes #75778.

Motivation: #75778 (comment).

It fixes a bug from net 7, so will need backporting.

Author:	ilonatommy
Assignees:	ilonatommy
Labels:	arch-wasm, area-Debugger-mono
Milestone:	7.0.0

[ilonatommy]

/backport to release/7.0-rc2

[github-actions]

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

[thaystg]

If the tests are passing, LGTM.

[ilonatommy]

/backport to release/7.0-rc2

[github-actions]

Started backporting to release/7.0-rc2: https://github.com/dotnet/runtime/actions/runs/3091794256

[radical]

@ilonatommy just curious, is VS using the privateProperties part?

[radical]

Changes look fine to me. I just have a few questions:

• How is VS surfacing the public/private/protected members? Is it reading anything other than "result"?
• We should also test this same thing with protected members being overridden by a derived class. And the base/derived split should be tested for being in the user code, and being base (!user code) -> derived(user code)

[radical]

Also, it would be a good idea to remove all the __* fields that we add to the json for our consumption, before sending them out from the proxy. The tests shouldn't depend on any of those.

[ilonatommy]

@ilonatommy just curious, is VS using the privateProperties part?

It is using it in the meaning: they are displayed. However, the difference between them and result is not stressed in VS - in Browser it is. To visualise, here are two screenshots from the same class in VS and Browser:

[ilonatommy]

Also, it would be a good idea to remove all the __* fields that we add to the json for our consumption, before sending them out from the proxy. The tests shouldn't depend on any of those.

I agree and Thays already mentioned this idea as well. The PR with it will be out soon.