Revert "Workaround FirstChanceExceptionEventArgs being trimmed"

[akoeplinger]

Reverts #6953

The fix for this issue is backported to dotnet/runtime 6.0 now so we can revert the workaround.

[akoeplinger]

@jonpryor ping

[jonpryor]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[jonpryor]

@akoeplinger: sorry for the delay.

Do you have a link handy to the dotnet/runtime commit which contains the fix?

[akoeplinger]

@jonpryor dotnet/runtime#68235 for main and dotnet/runtime#68265 for the 6.0 backport

[jonathanpeppers]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[jonathanpeppers]

LGTM, I reran two tests that had some failures.

[jonpryor]

Windows build stopped building. Rebased on main, force-pushed.

[jonathanpeppers]

This is good for .NET 7/main. Just FYI, we should be wary backporting this to .NET 6 -- maybe we don't even need to.

We'll need to make sure the dotnet/runtime schedule matches up with ours, if we do backport.

[akoeplinger]

Yeah backporting to .NET6 isn't really needed at this point I think since the workaround works and the only impact is the FirstChanceExceptionEventArgs type being preserved unconditionally rather than only when it's used.