Disabling Linux Bionic tests that fail on staging. (#72256)

* Disabling LinuxBionic tests that fail on staging.

* Attempt 2.

* Fix 3 - partial, w/o SpecialFile_Unelevated_Throws

Fix unsigned indexing with negative value on EnC (#72255)

FixContextForEnC was treating an index that may be negative as an
unsigned value. This would crash the runtime during EnC when running as
64-bit.

Fix #70834

Differentiate BF disabled error messages (#72248)

Fixes for NonBacktracking NFA mode (#72199)

* Fix NFA mode backtracking simulation

* Refactor to StateFlags

* Fix bug in timeout check

* Changes from review

Review error messages in JSON (#72254)

* Review error messages in JSON

* Clarify SerializerContextOptionsImmutable

* Update src/libraries/System.Text.Json/src/Resources/Strings.resx

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Text.Json/src/Resources/Strings.resx

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>

Add diagnostic suppressor for the MarkMethodsAsStatic diagnostics on custom marshallers (#72129)

[RateLimiting] Rename resourceID in PartitionedRateLimiter (#72214)

Fix most SafeHandle finalization occurrence in System.Net.Quic (#72221)

* Fix most SafeHandle finalization occurrence in System.Net.Quic

All of the remaining ones are coming from SslStreamCertificateContext.

* Apply suggestions from code review

Co-authored-by: Marie Píchová <11718369+ManickaP@users.noreply.github.com>
* Update src/libraries/System.Net.Quic/tests/FunctionalTests/QuicStreamTests.cs

Co-authored-by: Marie Píchová <11718369+ManickaP@users.noreply.github.com>
Co-authored-by: Marie Píchová <11718369+ManickaP@users.noreply.github.com>

Use a stackalloc for short SHA1 hashes

Disable caching in find_program (#72250)

Unblocks #71725.
Followup: #71742

Prior to #71742, we were calling `locate_toolchain_exec` once per tool. Calling it twice with different prefix gives us the old result, which is something we don't want (we only call this local function a few times in this file, and each time for intention to "lookup" the executable).

Add test for receiving H3_REQUEST_REJECTED from server (#72253)

Annotating libraries with small number of warnings to be AOT safe (#72108)

* Annotating librries with small number of warnings to be AOT safe

* FB

* Fix RR linux arm64 failure and remove ComponentModel annotation

Tweak SafeHandle finalization logging to include a count (#72223)

Helps to understand the impact a certain change makes and also to more easily refer to a given log entry.

link ignored test back to filed issue (#72188)

Enabling 9 new libraries tests for NativeAOT (#72167)

[wasm] Add interop code to the browser template (#72172)

* [wasm] Add interop code to the browser template

* Try to use AnalyzerReference instead of ProjectReference

* Try to make the analyzer reference unconditional

* fix

* no namespace fix

* InstallWorkloadFromArtifacts: Update the targeting pack from the local

.. nugets.

* Wasm.Build.Tests: remove the app.ref fixup from the targets

* workload-testing.targets: Ensure that app.ref nuget is available

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

Recognize TrimMode=partial in ILCompiler targets (#71117)

This should root all non-trimmable assemblies, same as TrimmerDefaultAction=copy.

We still support consuming ILCompiler through NuGet when targeting .NET 6, so recognize the old thing too. We might want to cut the .NET 6 support close to RC and at that time TrimmerDefaultAction handling can go away.

Related: https://github.com/dotnet/linker/pull/2856.

Update UserTypeMarshallingV2.md to match approved design (#72230)

[wasm] Set TrimMode=partial by default, which is equivalent to the older (#72234)

.. `TrimMode=link`.

The `TrimMode` values changed in https://github.com/dotnet/sdk/pull/26246 .

Fix chain cleanup in SslStream.SelectClientCertificate (#72222)

The chain's elements are being disposed of _after_ the chain itself is being disposed of, which means none of the elements are actually being disposed (chain.Dispose nulls out the collection such that accessing the property again lazily initializes it to an empty one).

wasm, js interop ownership (#72215)

Remove hardcoded non trimmable assemblies in oob.proj (#72171)

* Remove hardcoded non trimmable assemblies in oob.proj

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

Use the `TargetPathWithTargetPlatformMoniker` to flow through the IsTrimmable property that is set by projects and use that for illink validation assembly selection.

* Update oob.proj

* Update illink.targets

Remove IsPreRelease msbuild property (#72169)

* Remove IsPreRelease msbuild property

The property was added in https://github.com/dotnet/coreclr/commit/809b8f792923c95b9d5fee824c60dde21079cca1 years ago and isn't used anymore. Remove it to ease branding.

* Update configureplatform.cmake

Make llvm-objcopy preferred; no required (#72208)

Fix tar checksum calculation bug (#72139)

* Merge await using declarations with usages.

* Rename TarEntry test files.

* Fix tar checksum calculation bug

* Revert argument rename

Co-authored-by: carlossanlop <carlossanlop@users.noreply.github.com>

Use open images in HTTP/SSL pools (#72218)

* Use open image in http.yml

* Use open image in ssl.yml

Allow marshaller entry-point type to be a struct (#72207)

Use u8 in a few more places (#71992)

Fix more success path finalization in System.Security.Cryptography (#72203)

Finalizable objects being left for finalization on success paths

Fix a broken dprintf (#72198)

Fix stackoverflow issue when creating recursive contracts using JsonIgnoreCondition (#72186)

* Fix stackoverflow issue when creating recursive contracts using JsonIgnoreCondition

* fix formatting

Re-enable releaseondetach profiler test (#72144)

Refactor segment size computation (#71178)

Co-authored-by: Maoni0 <maoni@microsoft.com>

Fix error handling for generic arguments constrained as Enums (#72141)

Fixes #71884

Update dependencies from https://github.com/dotnet/llvm-project build 20220711.1 (#72162)

runtime.linux-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-musl-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.osx.11.0-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.win-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.win-x64.Microsoft.NETCore.Runtime.ObjWriter
 From Version 1.0.0-alpha.1.22252.1 -> To Version 1.0.0-alpha.1.22361.1

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

Implement HttpProtocolException for HTTP/3 (#72095)

* Replace Http3ProtocolException with HttpProtocolException

* Correctly process incoming protocol errors

* Fix HTTP3 stress

* Add some tests

* Remove formating changes

* Code review feedback

* Fix build

* Throw HttpProtocolException from content stream as well

* Add test for throwing when reading the content stream

Generalize init block morphing a little (#70899)

* Generalize init block morphing a little

Make it handle arbitrary local destinations.

* Add a test

Fix incorrect buffer reallocation in AcceptSecurityContext (#72184)

[wasm] JSImport feedback (#72174)

- fix exception marshaling on JSExport
- fix export class with no namespace on JSExport
- bind thisArg of function to closest scope in JSImport
Co-authored-by: Katelyn Gadd <kg@luminance.org>

Utilize new APIs more in System.Formats.Asn1

* Replace AsnWriter.Reverse with MemoryExtensions.Reverse
* Split BigInteger parsing and formatting between .NET Standard and Core where possible
* Use BinaryPrimitives in a few more places

Vector.Shuffle should use SSE2 for double overload when AVX is not availablee (#72168)

Don't explicitly link ICU into Mono (#70088)

We build against FreeBSD 12, with ICU 69. By adding an explicit dynamic link against the system ICU at build time, it means we can't execute on FreeBSD 13.1, with ICU 70. Removing the linkage doesn't seem to break anything (it should be pulled in via the ICU shim in System.Globalization), but it does allow the runtime to execute okay on FreeBSD 13.1.

[mono] Refactor UnmanagedCallersOnly_MonoAot tests (#71535)

* Removing UnmanagedCallersOnly_MonoAot due to duplication

* Reverting removal of UnmanagedCallersOnly_MonoAotTest

* Renaming the test to *Basic and reusing the native component

* Moving basic tests from the general test project into the basic one

* Utilizing source generator for test methods

Add missing inlining attribute (#72058)

ConfigurationBinder should support binding private properties from ba… (#71039)

* ConfigurationBinder should support binding private properties from base classes.

* Fix comment: consider binding 'Object' instance directly.

* Fix comment: directly use base-most methodInfo for properties; Add test cases for more get set combinations.

Remove an assert from impHWIntrinsic (#72084)

Get rid of hardcoded sleep in tracing/runtimeeventsource/nativeruntimeeventsource (#72010)

Fix WritePGOData (#72083)

We were constructing the TypeHandle before validating that the handle
did not represent an unknown handle. This crashes in the constructor in
debug builds due to some debug-only validation running there.

[wasm][debugger][firefox] Added exception handling for Firefox onEval (#72011)

* Enabled tests for Firefox.

* Add exception catching for Firefox.

Support caller-allocated buffer for stateless collection marshalling (#72122)

- Refactor blittable element handling into a base helper for shared functionality between stateless and stateful
- Enable using a caller-allocated buffer for stateless collection marshalling

[wasm][aot] Avoid making the 'this' variable volatile. (#71799)

This is only needed in non-llvmonly mode for computing stack traces in gshared methods.

Speed up ThreadLocalStatics_Test in DEBUG builds (#72151)

Saw this test timeout in https://github.com/dotnet/runtime/pull/72077. This should decrease the amount of work done by the test by 25%. It is the longest test in DynamicGenerics. It was only recently enabled.

```
Running Test: ThreadLocalStatics.TLSTesting.ThreadLocalStatics_Test

cmdLine:D:\a\_work\1\s\artifacts\tests\coreclr\windows.x64.Debug\nativeaot\SmokeTests\DynamicGenerics\DynamicGenerics\DynamicGenerics.cmd Timed Out (timeout in milliseconds: 1800000 from variable __TestTimeout, start: 7/13/2022 7:46:11 AM, end: 7/13/2022 8:16:11 AM)
Test Harness Exitcode is : -100
```

Use `-unicode` instead of `/unicode` for linux ildasm roundtrip tests (#71958)

Use `-unicode` instead of `/unicode` for linux ildasm roundtrip tests

Fix more SafeHandle cleanup in Windows crypto PAL (#72116)

* Fix more SafeHandle cleanup in Windows crypto PAL

Most of these are actually on the success path.  These were highlighted via the System.Net.WebSockets.Client tests with the new checked SafeHandle finalization flag.

* Apply suggestions from code review

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

Fix Tar stream seeking, flushing, and argument validation (#72045)

Fix all outstanding S.Linq.Expressions failures (#72077)

Product fixes:

* Don't try to be helpful in the exception when `MakeGeneric*` constraint validation fails. CoreCLR is not helpful and S.Linq.Expressions is testing for it.
* We need to inject delegate `Invoke` method dependencies elsewhere. A test looking to reflection-invoke the `Invoke` method was failing with `MissingMetadataException` but we have no right to be failing on that per policy.

Test fixes:

* Consider NativeAOT always trimmed. We could potentially do what Mono is doing, but we currently don't support substitution files (only embedded substitutions), and NativeAOT will always be trimmed.
* Mark a test that is fundamentally incompatible (and pointless) with trimming as such.
* Add a bunch of RD.XML.

Fixes #69944. Fixes #69928. Fixes #69943.

Mix newer kernel versions into library test matrix (#71740)

* Mix newer kernel versions into library test matrix
* Update eng/pipelines/libraries/helix-queues-setup.yml

Co-authored-by: Juan Hoyos <juan.hoyos@microsoft.com>
Co-authored-by: Juan Hoyos <juan.hoyos@microsoft.com>

Update DependencyGraphViewer to allow for DGML reading (#70690)

* Implement capability for users to upload .dgml files in addition to ETW events. Allows program to be run without admin permissions if the user has a .dgml file representing the dependencies in their program.

* Update UI to allow users to upload .dgml files from the file browser or the command line

* Update GUI to allow resizing of pages, clean up layout of pages

* Add test cases to ensure that graph is correctly replicated from sample .dgml files

* Restructure graph class to allow multiple edges and update GUI to show all reasons that two nodes are connected on one line

Co-authored-by: Aaron Robinson <arobins@microsoft.com>

Clean up runtime pack versioning logic (#72073)

ILASM - Disable `fileversionpreservation` test (#71972)

* Disable fileversionpreservation test

* Using <IlasmRoundTripIncompatible>true</IlasmRoundTripIncompatible> to disable roundtrip test for fileversionpreservation

* Update issues.targets

Polyfill the incremental generator ForAttributeWithMetadataName from roslyn (for LibraryImportGenerator). (#71652)

Co-authored-by: Jeremy Koritzinsky <jkoritzinsky@gmail.com>

Avoid unnecessary duplication of XxCng instances (#72120)

And also leaving some of the originals for finalization.

Fix handling of non-bidirectional MarshalMode.Default with collection marshalling (#72075)

Update dependencies from https://github.com/dotnet/arcade build 20220712.1 (#72093)

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 7.0.0-beta.22358.3 -> To Version 7.0.0-beta.22362.1

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

Port perf improvement changes in new API back to runtime. (#72109)

* Port perf improvement changes in new API back to runtime.

* Use normal type

If the base type is a sequential zero-sized type, treat a derived auto-layout type as starting from field offset 0. (#72065)

Make 'RC' lowercase in versions.props (#72110)

Fix style issue in X509ChainPolicy.Clone and add some tests

Add support for building WASM variants alongside single-threaded to the build system (#71054)

This change adds 2 additional wasm runtime packs: multithreaded and perftrace.

JIT: Ensure object derefs do not remove previous conditions in loop cloning (#72098)

Adding object dereferences was calling EnsureBlockConditions which would
reallocate all the lists, removing previous array dereferences. This
resulted in missing array null checks in some cases.

Fix #71611

Fix issue where the code fixer is not preserving the leading and trailing trivia for the regex generator (#72042)

Fix typeloader error messages for invalid byref fields (#72102)

Fixes #72090

Co-authored-by: Stephen Toub <stoub@microsoft.com>

JIT: Disqualify delegate calls to instance methods on value classes from GDV explicitly (#72089)

This was relying on the profile data containing the unboxing stub before.
That really only works out for dynamic PGO. Static profiles or textual
PGO data will end up with the exact method and hit a JIT assert passing
a TYP_REF this arg to a TYP_BYREF this param.

[main] Update dependencies from dotnet/icu dotnet/emsdk (#72094)

* Update dependencies from https://github.com/dotnet/icu build 20220712.2

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 7.0.0-preview.7.22361.2 -> To Version 7.0.0-rc.1.22362.2

* Update dependencies from https://github.com/dotnet/emsdk build 20220712.2

Microsoft.NET.Workload.Emscripten.Manifest-7.0.100
 From Version 7.0.0-preview.7.22361.2 -> To Version 7.0.0-rc.1.22362.2

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

[wasm][aot] Make the gc pin area smaller. (#71807)

[main] Update dependencies from dotnet/hotreload-utils (#72014)

* Update dependencies from https://github.com/dotnet/hotreload-utils build 20220711.1

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 1.1.0-alpha.0.22354.2 -> To Version 1.1.0-alpha.0.22361.1

* Update dependencies from https://github.com/dotnet/hotreload-utils build 20220712.1

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 1.1.0-alpha.0.22354.2 -> To Version 1.1.0-alpha.0.22362.1

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

[QUIC] System.Net.Quic API made public (#72031)

* System.Net.Quic removed from ASP transport package and made part of SDK ref

* Removed manual references to System.Net.Quic.csproj

[QUIC] API QuicStream (#71969)

* Quic stream API surface

* Fixed test compilation

* Fixed http test compilation

* HttpLoopbackConnection Dispose -> DisposeAsync

* QuicStream implementation

* Fixed some tests

* Fixed all QUIC and HTTP tests

* Fixed exception type for stream closed by connection close

* Feedback

* Fixed WebSocket.Client test build

* Feedback, test fixes

* Fixed build on framework and windows

* Fixed winhandler test

* Swap variable based on order in defining class

* Post merge fixes

* Feedback and build

* Reverted connection state to pass around abort error code

* Fixed exception type.

Support top-level statements in regex analyzer (#72046)

* Support top-level statements in regex analyzer

* Apply test suggestion

* Address feedback

Backport action: fix link to set org membership visibility (#72005)

We need to target the GitHub org that owns the repo, otherwise the collaborator check API doesn't work (i.e. check for dotnet instead of the Microsoft org).

Fix solution files via slngen (#72059)

* Remove System.Private.Runtime.InteropServices.JavaScript from solutions

* slngen

[wasm] Re-enable `System.Text.RegularExpressions.Tests.csproj` (#72076)

The linker bump in https://github.com/dotnet/linker/pull/2889 should
have fixed the issue.

Closes https://github.com/dotnet/runtime/issues/71848 .

More reliably clean up more SafeHandle instances (#71991)

* More reliably clean up more SafeHandle instances

This calls Dispose on SafeHandles (or things wrapping SafeHandles) that were otherwise being left for finalization:
1. Some of these are fixing finalization happening even on success paths (typically where the implementation isn't disposing of something that directly or indirectly wraps a SafeHandle).
2. Some of these are fixing finalization happening for invalid SafeHandles.
3. Some of these are fixing tests to finalize less.  My goal with fixing the tests was to eliminate the noise in order to find instances of the other two cases.

These were found primarily via two means:
- Debug-instrumentation in SafeHandle to log when one is finalized.  This instrumentation is built into debug/checked builds of SafeHandle and requires setting the DEBUG_SAFEHANDLE_FINALIZATION environment variable to "1".
- Auditing use of SafeHandle.IsInvalid

There's a lot more that can be cleaned up using the SafeHandle instrumentation, but I'm pausing here for now.  The System.IO.Pipes, System.IO.FileSystem, and System.Security.Cryptography tests are clean on Windows.

Fix ConsoleStream.Flush to not throw for stdin (#72035)

* Fix ConsoleStream.Flush to not throw for stdin

Stream.Flush{Async} is supposed be a nop on a read-only stream.

* Disable new test on unsupported platforms

Mark JsonSerializerOptions.TypeInfoResolver as nullable and linker-safe (#72044)

* Mark JsonSerializerOptions.TypeInfoResolver as nullable and linker-safe

* Update src/libraries/System.Text.Json/tests/System.Text.Json.Tests/Serialization/OptionsTests.cs

* Ensure TypeInfoResolver value is always populated in JsonSerializerOptions.Default

[QUIC] API QuicConnection (#71783)

* Listener comment; PreviewFeature attribute

* Feedback

* QuicConnection new API including compilable implementation

* Fixed logging

* Fixed S.N.Quic and S.N.Http tests

* Options now correspond to the issue

* Feedback

* Comments, PreviewFeature attribute and RemoteCertificate disposal.

* Preview feature attribute is assembly wide

* Some typos

* Fixed test with certificate

* Default values as constants

* Event handlers split into methods called via switch expression.

* Some more comments

* Unified unsafe usage

* Fixed some more tests

* Cleaned up some exceptions and resource strings.

* Feedback

* Latest greatest API proposal.

* Fixed Http solution

* Feedback

Bump main version to RC1 (#72068)

Co-authored-by: carlossanlop <carlossanlop@users.noreply.github.com>

[wasm][debugger] Show correct and complete function names on callstack (#72048)

* Fixing 71553

* Fixing merge.

* Addressing @radical comments.

* Addressing @radical comments

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

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

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

Co-authored-by: Ankit Jain <radical@gmail.com>
* Remove console.writeline

* Fix compilation error

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

ILASM - Allow EntryPoint methods to be on interfaces (#71975)

[main] Update dependencies from dotnet/linker (#72063)

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22362.1 -> To Version 7.0.100-1.22362.3

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

Properly report `TypedReference` with `ref` fields. (#72062)

* Update details about ELEMENT_TYPE_TYPEDBYREF

Details in siginfo.cpp were stale.
Add some logging that helped track down the issue.

* Fix byref reporting in crossgen2

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

Centralize calling convention handling (#71999)

The newstyle calling convention handling being exclusive to JitInterface is causing problems in other spots of the AOT compiler where we would need to replicate the rules of when to read custom modifiers, which ones matter, what's their relationship to `MethodSignatureFlags`, and what combinations of the modifiers makes sense.

I'm adding a single `UnmanagedCallingConventions` enum that captures all the calling conventions we support and add means to get to this from p/invoke, `UnmanagedCallersOnly` method and from a calli signature.

[RateLimiting] Rename WaitAsync and RateLimitPartition methods (#72054)

Add a fabricbot task to add issues to the AppModel project (#71672)

Add predefined cpu names for --instruction-set (e.g. haswell) (#71911)

Provide upgrade response details (#71757)

* Provide Upgrade response details

* fixing tests

* Address review feedback

* Save HttpStatusCode without CollectHttpResponseDetails

* Remove unnesessary skip on test

* Disable ConnectAsync_Failed on browser since CollectHttpResponseDetails is not supported

* Update src/libraries/System.Net.WebSockets.Client/tests/ConnectTest.cs

Co-authored-by: Natalia Kondratyeva <knatalia@microsoft.com>
* Address review feedback

* Revert "Save HttpStatusCode without CollectHttpResponseDetails"

This reverts commit 0713bd8e292b6a76b0b9f297d95e466f11feff3b.

* renove using from ref

* Update test

* Update src/libraries/System.Net.WebSockets.Client/src/System/Net/WebSockets/ClientWebSocket.cs

Co-authored-by: Natalia Kondratyeva <knatalia@microsoft.com>
* fixing Values and Enumerator for HttpResponseHeaders

* fixing Values and Enumerator for HttpResponseHeaders

* Apply suggestions from code review

Co-authored-by: Miha Zupan <mihazupan.zupan1@gmail.com>
* Update src/libraries/System.Net.WebSockets.Client/src/System/Net/WebSockets/WebSocketHandle.Managed.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Apply suggestions from code review

Co-authored-by: Miha Zupan <mihazupan.zupan1@gmail.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Check CollectHttpResponseDetails setter

* disable CA1822 // Mark members as static

Co-authored-by: Natalia Kondratyeva <knatalia@microsoft.com>
Co-authored-by: Miha Zupan <mihazupan.zupan1@gmail.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

PropertyVisibilityTests.cs: Fix invalid change in tests merged earlier (#72043)

Add the approved span marshallers and remove the v1 model (#71989)

* Add SpanMarshaller and ReadOnlySpanMarshaller. Move the new attribute types to System.Runtime ref assembly

* Remove pinning of the managed type. The new model isn't supposed to support this and it was breaking the span marshallers.

* Delete the V1 marshalling model

You served us well, but your time has passed.

* Remove obsoleted unit tests. Rename type usages in JS generator

* Fix test build

* PR feedback and add doc comments.

* Fix some extraneous usages of the v1 API in tests.

* Remove test of the managed type's GetPinnableReference method. We don't support this with the v2 system.

Improve argument validation for Rfc2898DeriveBytes.

Add CertificateChainPolicy to ssl options

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