[MonoAOTCompiler] accept more than one AotProfilePath (#57111)

* [MonoAOTCompiler] accept more than one AotProfilePath

Fixes part of: https://github.com/dotnet/runtime/issues/56989

The Android workload needs to be able to pass in multiple AOT profiles
to the `<MonoAOTCompiler/>` MSBuild task. We plan on shipping a
default profile in the Android workload, as well as the MAUI workload.
.NET MAUI would have two profiles when using profiled AOT.

Adding `profile-only`, and then multiple `profile` arguments is
currently working in legacy Xamarin.Android:

https://github.com/xamarin/xamarin-android/blob/77895e2a03ed91fdf3729cde54501f91e7d1a36f/src/Xamarin.Android.Build.Tasks/Tasks/GetAotArguments.cs#L256-L260

Either a single property or item group can be passed in for
`AotProfilePath` now. I avoided changing the name, as it looks like it
might be used by wasm.

* Update src/tasks/AotCompilerTask/MonoAOTCompiler.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
Co-authored-by: Ankit Jain <radical@gmail.com>

Improve docs for BinaryData.ToString() (#56325)

* Improve docs for BinaryData.ToString()

* Apply suggestions from code review

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>

Make sure rid specific libs projs get packaged (#57193)

* Make sure rid specific libs projs get packaged

RID specific runtime.*.runtime.native.System.IO.Ports projects weren't
packaged because the GeneratePackageOnBuild property wasn't set to true
for them. The property was only true during an allconfigurations build.
As these projects are only built outside of an allconfigurations build,
the GeneratePackageOnBuild property needs to account for such as well.

Also updating the NoTargets Sdk to clean the rid specific package up.

Moving the GeneratePackageOnBuild logic into a props file and set it to
false during servicing (or runtimelab) so that devs can set the property
during servicing directly in the project file if a project should be
packaged.

* Remove the GeneratePackage property

That property was intended to enabled incremental servicing but with
now using the NuGet Pack task we can just make use of the publicl
available GeneratePackageOnBuild property.

* Fix IsPackable setting

Fix regression in dictionary key serialization when registering custom primitive converters. (#57138)

* Fixes regression in dictionary key serialization when registering custom primitive converters.

* reinstate debug assertion

* add test case for JsonTypeInfo overloads

* update method names

* fix sourcegen test

[interp] Add wrapper for calli to native pointer (#57119)

If the signature has suppress gc transition attribute, we don't emit the wrapper and call native code directly.

Annotate unsupported APIs in System.Diagnostics.Process (#57120)

[MacCatalyst][libraries] Update MacCatalyst as Case Preserving for tests (#57167)

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>

Callconv matching in PassingBy{Out,Ref}Test (#57075)

Signed-off-by: Timur Mustafin <t.mustafin@partner.samsung.com>

Support NoGC for regions (#54085)

add timeout handling to QuicStream (#56060)

* add timeout handling to QuicStream

* feedback from review

* fix bad resolve

* feedback from review

Fix perms on diagnostic files (#57177)

Optimize string.join for IList<string> with char separator (#56857)

* Improve string join for char separator

* Feedback

Refactor ManagedWebSocket to avoid forcing Task allocations for ReceiveAsync (#56282)

* Refactor ManagedWebSocket to avoid forcing Task allocations for ReceiveAsync

The ManagedWebSocket implementation today supports CloseAsyncs being issued concurrently with ReceiveAsyncs, even though CloseAsync needs to issue receives (this allowance was carried over from the .NET Framework implementation).  Currently the implementation does that by storing the last ReceiveAsync task and awaiting it in CloseAsync if there is one, but that means multiple parties may try to await the same task multiple times (the original caller of ReceiveAsync and CloseAsync), which means we can't just use a ValueTask.  So today asynchronously completing ReceiveAsyncs always use AsTask to create a Task from the returned ValueTask.  This isn't actually an additional task allocation today, as the async ValueTask builder will create a Task for the asynchronously completing operation, and then AsTask will just return that (and when it completes synchronously, there's extra code to substitute a singleton).  But once we switch to using the new pooling builder, that's no longer the case.

This PR uses an async lock as part of the ReceiveAsync implementation, with the existing async method awaiting entering that lock.  CloseAsync is then rewritten to be in terms of calling ReceiveAsync in a loop.  This also lets us remove the existing Monitor used for synchronously coordinating state between these operations, as the async lock serves that purpose as well.  Rather than using a SemaphoreSlim, since we expect zero contention in the common case, we use a simple AsyncMutex that's optimized for the zero contention case, using a single interlocked to acquire and a single interlocked to release the lock.

* Fix misleading comment

Fix wrong line info when debugging CoreLib in windbg (#57176)

Issue: https://github.com/dotnet/runtime/issues/52774

Changed couple of the DAC functions used to use the code version manager to
get the properly native code start address.

PhysicalFileProvider: Remove second try on GetFileLinkTargetLastWriteTimeUtc  (#57136)

* return the value from 2nd try

* Do not attempt a second try and just capture the FileNotFoundException

* Update src/libraries/Microsoft.Extensions.FileProviders.Physical/src/Internal/FileSystemInfoHelper.cs

Co-authored-by: David Cantu <dacantu@microsoft.com>
Co-authored-by: Dan Moseley <danmose@microsoft.com>

Disable 2 memory heavy array tests for 2 distros (#57108)

* Disable 2 Array tests on 2 distros

* Update string

Clean up JSON src-gen known-type resolution (#57092)

Unblock publishing symbols with exclusion file. (#57162)

Avoid dereferencing target pointer (#57164)

Pass runtime flavor args when copying native test components in CI (#57073)

* Pass runtime flavor args when copying native test components in CI
* Make test build script pass RuntimeFlavor property

Add runtime capability to allow updating parameters (#56599)

Update helix version (#57084)

Get better assert failure in flaky FlushAsync_ThrowsIfWriterReaderWithException test (#57081)

Update dependencies from https://github.com/mono/linker build 20210809.3 (#57124)

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.6.21405.1 -> To Version 6.0.100-preview.6.21409.3

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

Switch to using globalconfig instead of ruleset for analyzer configuration (#57144)

Fix COM test failures on Windows Nano Server (#57148)

* Disable ComWrappers activation tests when running on Windows Nano Server since they require reg-free COM.

* Run the test on a separate STA thread so we can get far enough in Main to detect running on nano server before we use the STA.

Delete dead System.Net.Connections code (#57129)

This never shipped, and won't in .NET 6.  If we decide to bring it back for a future release, it'll involve changes, and we can get whatever base code we want to start with from git history.  We don't need all of this to end up in an LTS branch.

Prevent invoking property 'init' accessor from Visual Basic late binder (#57025)

JIT: fix gc type for PUTARG_SPLIT codegen (#57099)

If we need to swap registers during codegen for PUTARG_SPLIT, make sure
we use the right GC type.

Fixes #13127.

Update the runtime code where iOS/MacCatalyst logic is currently used to benefit from the updated analyzer behavior (#57072)

* Update the runtime code where iOS/MacCatalyst logic is currently used to benefit from the updated analyzer behavior

Don't log an error when a BackgroundService is canceled due to the host being stopped. (#57005)

* Don't log an error when a BackgroundService is canceled due to the host being stopped.

Fix #56032

* Add volatile to stopping field.

* Convert HostTests to use a logger instead of depending on EventSource.

* Make it obvious the test is using the default worker template

Check `EventSource.IsSupported` in code paths specific to thread pool worker tracking (#56862)

Check `EventSource.IsSupported` in code paths specific to thread pool worker tracking

Fixes https://github.com/dotnet/runtime/issues/47871

generic math: Don't include System.Runtime reference assembly when publishing (#57071)

System.Runtime.Experimental.csproj builds a reference assembly called System.Runtime.dll that should be used when compiling, but should not be included when packaging. For normal out of tree projects this happens because the System.Runtime.Experimental.nupkg makes it evident that <PackageReference Include="System.RuntimeExperimental" /> only provides reference assembly assets. But in-tree for <ProjectReference> we have to be explicit.

Fixes https://github.com/dotnet/runtime/issues/55823
Fixes https://github.com/dotnet/runtime/issues/55767
Fixes https://github.com/dotnet/runtime/issues/56836

Handle FileNotFound for symlinks when using polling (#56915)

* Handle FileNotFound for symlinks when using polling

* Re-enable tests

* Use CTS instead of Task.Wait(TimeSpan) to fix issues in browser.

[libraries][iOS][tvOS] Reenable System.Resource.ResourceManager and System.Collections.Immutable tests (#56994)

* [libraries][iOS][tvOS] Reenable System.Resorce.ResourceManager.Tests tests

* [libraries][iOS][tvOS] Remove ActiveIssue from test with no test data

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>

Fix ServiceController test (#57116)

Disable GetBitLengthLarge for 32 bit due to OOM (#57110)

Fix analyzer, docs file packaging and don't overbuild runtime.native IO.Ports packages. (#57118)

* Fix analyzer and docs file packaging

This commit fixes analyzers which were dropped from the package during
the clean-up of pkgprojs and missing documentation files.

1. Analyzers were dropped from the package because the BeforePack
   hook wasn't respected. This happens when the NuGet Pack nuget package
   is referenced which then gets imported before the packaging.targets
   file. The BeforePack property needs to be set in a props file if
   NuGet's targets file isn't used from the SDK but from its package.
2. Documentation files were dropped as the
   DefaultAllowedOutputExtensionsInPackageBuildOutputFolder property
   didn't include the .xml extension.

* Don't overbuild runtime native IO packages

Re-add S.T.J.SourceGeneration.UnitTests to STJ sln (#57080)

* Re-add S.T.J.SourceGeneration.UnitTests to STJ sln

* Update slngen.template.proj

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

Apply redundant branch optimizations in post-order of dominators (#57074)

In #56979 we end up with outdated dominators and make a wrong decision
based on it. To avoid this situation, visit basic blocks in post-order
of the dominator tree to ensure that we have up-to-date dominators for
each basic block.

Fix #56979

Fix performance for `File.GetLastWriteTimeUtc` on Unix (#53070)

All this changes are to internal methods which are later called by other public methods which correctly call `ToLocalTime` for local time "overloads".
Performance hit is most visible for application which doesn't need to load local timezone information which takes ~30ms on 2016 MacBook Pro.

Co-authored-by: Jeff Handley <jeff.handley@microsoft.com>

Fix ThreadLocal tracking behavior (#56956)

- Before this change the trackAllValues behavior for ThreadLocal<SomeParticularType> was defined by the first instance of thread local to have its value set on the thread
  - This could lead to unpredictable memory leaks (where the value was improperly tracked even though it wasn't supposed to be) This reproduces as a memory leak with no other observable behavior
  - Or data loss, where the Values collection was missing entries.
- Change the model so that ThreadLocal<T> trackAllValues behavior is properly defined by the exact ThreadLocal<T> instance in use
- Implement by keeping track of the track all changes behavior within the IdManager

Fixes #55796

Automatically generate .NETStandard compat errors (#57057)

* Automatically generate .NETStandard compat errors

Before this change, a project had to explicitly add a
NETStandardCompatError item to declare that a given tfm range
shouldn't be supported when packaging.

Automate this logic so that projects don't need to specify the item
themselves anymore. Instead condition the NETStandardCompatError target
calcuation logic on the existence of both a .NETStandard and a
.NETCoreApp tfm.

* Update Directory.Build.targets

* Update packaging.targets

* Update eng/packaging.targets

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>
Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>

add more validation to quic config (#56923)

Fix half bug (#57047)

* Fix half bug

* parens

Remove unused System.Reflection.Emit leftovers (#57069)

Delete old AssemblyExtensions.ApplyUpdate/GetApplyUpdateCapabilities (#56511)

* Delete old AssemblyExtensions.ApplyUpdate/GetApplyUpdateCapabilities

* Fix a few missed uses in mono tests

Fix typo in CreateDefaultPropagator doc comment (#57103)

Improve GUI error dialog when no runtime is installed (#57089)

* Add architecture and runtime version to error dialog

* PR feedback

Clean up ComWrappers tests (#57015)

* Remove unnecessary defines

* Clean up managed ComWrappers tests

Workaround exception filter linker issues with Http3RequestStream (#57011)

The type of exception filters used cause the linker to generate invalid IL when building the AOT functional tests for iOS. This in turn causes the mono aot compiler to issue a warning about the invalid IL AND fails the build.

Addresses #57010 and unblocks failing builds on CI

Fix auto-atomicity for loops around required one/notone/set loops (#56735)

Given an expression like `(?:a{2,3}){2}`, we currently erroneously convert that to `(?>a{2,3}){2}`, i.e. making the inner loop atomic, even though that then causes this to fail to match `aaaa` when it should match it (it fails because the first iteration will atomically match `aaa` and thus won't given any back, and then the next iteration will fail to match at least two `a`s).

The simple fix is in FindLastExpressionInLoopForAutoAtomic, removing the special-case that tries to make the body of a loop that's a one/notone/set loop atomic.  There are likely special-cases of this special-case that are still valid, but I'm not convinced it's worth trying to maintain and risk other gaps.

Update RequiresPreviewFeatures attribute (#56938)

* Add constructors and Message, URL properties

* Update attribute and tests

* sq

* Address feedback

* Just adding docs

[interp] Add IL seq points (#56137)

* [interp] Generate il seq points

Interpreter was generating only sdb seq points for implementing debugger support. Add also il seq points which are used for stack traces when debugging information is not present. We add IL seq points in locations where IL stack is empty, same as JIT.

* [interp] Use interp_prev_ins in more places

It skips over NOPs

* [tests] Disable all tests in readytorun folder on mono

We were disabling some of them all over the place. They shouldn'd be running on mono.

[mono] Fix conversions from float to integer (#57048)

* [interp] Fix conversions from float to integer

.un prefix is ignored for these conversions

* [mini] Fix conversions from float to integer

.un prefix is ignored for these conversions

* Re-enable test

Recognize VS2022 with CMake (#57063)

Fixes #57062

Fix building CoreCLR for x86 with the Windows 10.0.20348.0 SDK (#57067)

The SDK now defines CONTEXT_UNWOUND_TO_CALL in more cases, so we get a macro redefinition error. Since the SDK defines it to the same value as we do, just skip our definition if it's already defined.

MonitorEnter test needs to handle the case of the threadpool cleaning up other threads (#57008)

* MonitorEnter test needs to handle the case of the threadpool cleaning up other threads
- On rare gcstress test occasions there can a thread freed after allocating the 1024 threads but before the highThread is allocated

* Fix comment

More JSON xml updates (#57022)

Do not run crossgen2 under GC stress in tests (#56949)

[libraries][Android] Remove ActiveIssue from BinaryFormetter dependent System.Collections test (#56985)

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>

[libraries][Android] Reenable System.Collections.Immutable Tests (#56988)

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>

Fix missing offset in contained BITCAST for STORE_LCL_FLD (#57059)

Strong name key id cleanup and pkg testing move (#57044)

* Move src/libraries/pkg/test to src/libraries/testPackages

* Remove default StrongNameKeyId Open properties

Ensure Directory.Delete works fine even if user has no ListDirectory permissions (#56996)

One more place to take care for better error handling (#56997)

* One more place to take care for better error handling

* Handle null

* Rename function

Remove salt length check from Rfc2898DeriveBytes

Some standards/protocols want salts smaller than we permit, so we're really just getting in people's way.

Remove unused System.Reflection.Emit leftovers (#56698)

* Remove unused System.Reflection.Emit leftovers

Follow up to #56153

* Remove nCreateISymWriterForDynamicModule

* Removes LineNumberInfo, MarkSequencePoint & GetMethodSigHelper

* Remove MarkSequencePoint from mono

* Fix GetMethodSigHelper overloads

* Match changes in mono

* Remove SequencePointList from mono

Remove executable bit from unrelated files (#57051)

[wasm] Skip tests for ExpectContinue HTTP header (#57058)

Skip tests for ExpectContinue HTTP header for WASM

Fix reference preservation for boxed structs (#56412)

* Fix reference preservation for boxed structs.

* add null assertion for BoxedStructReferenceId when writing a new instance

Fix AccessViolation/InvalidProgramException in custom nullable converters. (#56577)

* Fix AccessViolation/InvalidProgramException in custom nullable converters.

* address feedback

* add comments to new test converters.

Override memory-based ReadAsync & WriteAsync for SerialStream.Unix (#56866)

Move corehost.proj to corehost directory (#57036)

* Move corehost.proj to corehost directory

* Simplify host version overriding

Fix invalid CodeDom comment generation when a new line starts with a slash (#56640)

* Failing tests where CodeDom creates invalid comment types

* Prevent CodeDom from creating invalid comment types

* Make C# generator impl more consistent with VB generator

* Switch attribute from ActiveIssue to SkipOnTargetFramework

* Be even more conservative about when the space is needed

* Attempt to improve readability

* Update VBCodeGenerationTests.cs

Co-authored-by: Dan Moseley <danmose@microsoft.com>

JIT: fix morph tail call copy opt alias analysis (#57009)

If there are non-call references to an implicit byref local, disqualify
that local from morph tail call copy optimization.

The fix is conservative in that "harmless" non-call references like field
accesses will also disqualify the opt. This can be improved on with extra
analysis in local morph.

Closes #56734.

Remove HasWildCardCharacters method (#57028)

It is not used any more.

JIT: arm64 redundant move peephole must check instruction format (#57016)

Otherwise we may mistake an immediate for a register and do an incorrect
optimization.

Fixes #56689.

JIT: don't update early ref counts in morph (#57037)

These updates are a vestige of an earlier time when we could not distinguish
early ref counts from normal ref counts. Updating them makes reasoning using
them during morph tricky, and there's no longer any need to do this.

Prerequisite to fixing #56743.

Link issue #10478 (#56728)

* Link issue #56726

* Update to 10478

Fix ARM64 floating point registers unwinding in PAL (#56919)

* Fix ARM64 floating point registers unwinding in PAL

We were not unwinding the non-volatile floating point registers at all
(not transferring them between the CONTEXT and ucontext_t before and
after the unw_step). That causes crashes on arm64 Unix in some of
the tests since JIT now generates code that uses e.g. the D8 register
and a runtime code that was throwing an exception was using it too.

* Fix non-OSX arm64, remove test csproj patch

* Fix Linux arm64

* Fix Linux arm64 - now really

Fixes #56522

Fix license in two code documents (#56755)

Additional Diagnostics in Dependency Injection (#56809)

* Additional Diagnostics in Dependency Injection

Log events when a ServiceProvider is created:

* How many singleton, scoped, transient services?
* Log the list of registrations

Fix #56313

* Add ServiceProvider HashCode to all events.

* Write ServiceProvider information when DependencyInjectionEventSource becomes enabled.

This allows for listeners to attach to a process after it is running, and get the DI information.

* Update new events to use Informational level and to have a Keyword.

* Switch to use WeakReference when holding on to ServiceProviders in DependencyInjectionEventSource.

[wasm] Redesign of managed objects marshaling and lifecycle (#56538)

file cycle of JS owned C# instances is driven by FinalizationRegistry

- got rid of Runtime._weakDelegateTable and Runtime._rawToJS
- got rid of JSObject.WeakRawObject and JSObject.RawObject
- GCHandle instead of JSObject double proxy for plain managed ref types
- GCHandle instead of int sequence for delegates + redesign of invocation
- GCHandle for task + redesign of invocation
- improved in-flight retention of thenable/promise and Task
- explicitly delegate type of parameter for EventListener
- moved and renamed some binding functions
- renamed all handles to jsHandle or gcHandle as appropriate
- removed jsHandle math
- cleanup of unused functions
- improved error messages for invalid handles
- more unit tests

Fixes for text-based PGO (#56986)

Fix missing indirection when reading in text-based PGO data.

Prioritize reading text-based PGO over dynamic or static PGO data, so that we
can use text to provide a fixed set of PGO data when trying to isolate bugs.

In the jit, give text-based PGO data the same trust level we give to dynamic
PGO data.

Sgen: Added 'default-namespace' argument (#46500)

* Sgen: Added an optional 'default-namespace' argument

The .NET runtime support loading pre-generated assemblies according to both the type & defaultNamespace when creating an XmlSerializer instance,
Given that, it is extremely useful to have the ability to generate an assembly for different defaultNamespace.

* Added defaultNamespace argument to XmlReflectionImporter

* Prevent setting default-namespace twice

* Added missing using statement

Fixes #56930 (#57000)

* Don't eliminate store for the following sequence of instructions
```
ldr wzr, [addrReg, #immOff]
str wzr, [addrReg, #immOff]
```

* Add regression test for https://github.com/dotnet/runtime/issues/56930

Disable test failing in .net framework (#56869)

* Disable test failing in .net framework

Change IOptionsSnapshot to reuse options instances across scopes (#56271)

* Change IOptionsSnapshot to reuse options instances across scopes

* Improve snapshot perf for unnamed option and delay dictionary alloc

Co-authored-by: Kahbazi <akahbazi@gmail.com>
Co-authored-by: Kahbazi <akahbazi@gmail.com>

Allow Multiple EventSources with same name (GUID) (#56873)

Fix contract violation in debugger func eval (#56933)

`DebuggerEval::DebuggerEval` is called from `Debugger::FuncEvalSetup` which is a NOTHROW function. As the `GetInteropSafeExecutableHeap` and associated `Alloc` functions can fail with an OOM, move the allocation out of the constructor to the caller where we can safely return `E_OUTOFMEMORY`.

This contract violation prevents debugging on chk and debug builds of the runtime.

Fixes #54504

[wasm][debugger] Fixing async locals in nested ContinueWith blocks (#56911)

* Adding test for #41984

* Adding old @radical tests and fixing it.

* Fixing tests.

* Update src/mono/wasm/debugger/BrowserDebugProxy/MonoSDBHelper.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
* Update src/mono/wasm/debugger/tests/debugger-test/debugger-async-test.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
* Addressing @radical comments.

* Addressing @radical comments.

* Addressing @radical comments.

Co-authored-by: Ankit Jain <radical@gmail.com>

Remove System.Linq in Blazor WASM apps (#56937)

Removes the last usages of System.Linq in a default Blazor WASM app, which were a handful of ToArray calls.

Fix #56916

This fixes a perf bug in revisit_written_pages for regions - if we are called with concurrent_p == FALSE, we start iterating at gen 0. This causes the small_object_segments to be set to FALSE when we are done with gen 0. This is of course incorrect, we should set this flag when we are done with gen 2. Fix is to modify the if-condition to test for the end of gen 2. (#56987)

More Extensions doc update (#56991)

[Mono]: Make EventPipe MonoProfiler provider GC events async safe. (#56714)

* Make EventPipe MonoProfiler provider GC events async safe.

EventPipe MonoProfiler provider includes most Mono profiler events,
but events currently triggered during GC is not emitted since EventPipe
is not async safe.

This commit add support to emit the MonoProfiler events triggered during
GC (gc resize, gc moves, gc roots and heap dump object references) into
a async safe temporary memory buffer that will be emitted into EventPipe
once GC completes (and world has been restarted). This opens up the
ability to do heapshots using EventPipe running on Mono into nettrace
files.

Heapshots can be triggered in same way as on CoreCLR, creating an
EventPipe session with GCHeapCollectKeyword keyword. Only one heapshot
can be triggered at a time, so in order to trigger an additional
heapshot, a new EventPipe session is setup (after the previous heapshot
has completed). Heapshot events will be included in all session with
configured keywords.

It is also possible to include all vtable/class references
(including class name), into heap dump. This is enabled by a specific
keyword and makes the dump self contained (no need to track type load etc).
That way runtime can start up without any profiler support, and then it
is possible to use custom tool or dotnet-trace to request a heap dump
(including all referenced vtable/class references) and once that
session is done, it is possible to do a new session and get a new
separate heap dump into a new session. Having separate sessions in
different files opens up the ability to diff between any heap dump over
time.

Reset loop numbers in RecomputeLoopInfo (#56981)

They become stale after optimizations.

Fix race condition in Net.Security telemetry test (#56984)

Refactor repeated invalid character checks in Path.GetFullPath Unix and Windows (#56568)

* Refactored repeated checks for invalid path chars

* Removed checkedForInvalids param and renamed method to GetFullPathInternal

* Apply suggestions from code review

* Cleaned up trailing whitespace

Co-authored-by: Jan Kotas <jkotas@microsoft.com>

Add JSON src-gen support for deserializing with parameterized ctors (#56354)

* Add JSON src-gen support for deserializing with parameterized ctors

* Address API review and PR feedback

* Fix issue with init-only properties

* Make misc fixes for build