[wasm][debugger] Removed internalProperties group.
Fixes https://github.com/dotnet/runtime/issues/75778.
Motivation: https://github.com/dotnet/runtime/issues/75778#issuecomment-1252541452.
It fixes a bug from net 7, so will need backporting.
Tagging subscribers to 'arch-wasm': @lewing See info in area-owners.md if you want to be subscribed.
Issue Details
Fixes https://github.com/dotnet/runtime/issues/75778.
Motivation: https://github.com/dotnet/runtime/issues/75778#issuecomment-1252541452.
It fixes a bug from net 7, so will need backporting.
| Author: | ilonatommy |
|---|---|
| Assignees: | ilonatommy |
| Labels: |
|
| Milestone: | 7.0.0 |
/backport to release/7.0-rc2
Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/3091753776
If the tests are passing, LGTM.
/backport to release/7.0-rc2
Started backporting to release/7.0-rc2: https://github.com/dotnet/runtime/actions/runs/3091794256
@ilonatommy just curious, is VS using the privateProperties part?
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 just curious, is VS using the
privatePropertiespart?
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:


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.