HTTP2: fix header corruption issue (dotnet/corefx#40487)

* HTTP2: fix header corruption issue

Fixes a bug in HPackEncoder which results in header
data corruption in cases where the write buffer has
not been zeroed out. Adds HPack roundtrip tests.
Fixes dotnet/corefx#40459.

Commit migrated from https://github.com/dotnet/corefx/commit/6f9796080563c9c40359e641e3287fabe42b580c

Move IsReadOnly check inside if-block to avoid cascading if-else (dotnet/corefx#40446)

Commit migrated from https://github.com/dotnet/corefx/commit/988783b4568682bf306875f28ae76e90e7da92a5

Fix unnecessary reliance on ValueTask being backed by a Task (dotnet/corefx#40511)

* Fix unnecessary reliance on ValueTask being backed by a Task

Arbitrary ValueTasks musn't be consumed twice.  Some code in WebSockets we doing so explicitly with the knowledge that the ValueTask it was consuming was actually safe in this pattern, but there's no need for that; with a minor restructuring, that can easily be avoided.

* Address PR feedback

Co-Authored-By: Gert Driesen <gert.driesen@telenet.be>
Commit migrated from https://github.com/dotnet/corefx/commit/cc4ceae30f14b69b625c5275315bbf873be84c01

Fix mail message encoding for transport when delivery method is pickup (dotnet/corefx#40492)

* Fix SmtpClient encoding message for transport when delivery method is pickup

* Reduce tested range of values to more closely bracket where error occurs

* Modify unit test to ensure line folds and validate result

Commit migrated from https://github.com/dotnet/corefx/commit/0de845b6baac154b27ff845774c467018b94c047

Add regression test for https://github.com/dotnet/corefx/issues/39770 (dotnet/corefx#40510)

Commit migrated from https://github.com/dotnet/corefx/commit/0b0c64c4302dfb1bc2a34ca2c1b5fbf9c5f609f6

Re-enable source link validation (dotnet/corefx#40507)

dotnet/corefx@fa168dd690798a683ef0d1e65d60ce5d6918d987 in arcade should have a fix to prevent OOM exceptions.

Commit migrated from https://github.com/dotnet/corefx/commit/4dae0831fccad7b81fc8c4ef187674edb052a1b3

Fix AsyncValueTaskMethodBuilder tests (dotnet/corefx#40505)

A few tests are intended to validate internal implementation details, and those are commented.  But the rest should be asserting implementation specifics.  This fixes those, and augments some validation a bit as well.

Commit migrated from https://github.com/dotnet/corefx/commit/ba9f27788bd99a29e5d832531f0ee86d8eae4959

Console tests: remove tests that expect to observe ANSI sequences (dotnet/corefx#40495)

These tests expect ANSI escape sequences to be written to the Console.SetOut stream,
which doesn't match with the implementation (since https://github.com/dotnet/corefx/commit/dotnet/corefx@98552707c585bc0f0c138fdeed991c28f00a0ac5).

Commit migrated from https://github.com/dotnet/corefx/commit/49a351478132e1c537f0cf3a02b77bbd232238f4

Re-enable xunit warning 2017 (dotnet/corefx#40467)

* Re-enable xunit warning 2017

Use specialized assertions for checking for elements in collections.

* Revert Re-enable xunit warning 2017, revert some test changes

* Address PR feedback

Commit migrated from https://github.com/dotnet/corefx/commit/2fc48e13e87a4e68576d7df7af420ac2874e929e

Ignore extension property if null. (dotnet/corefx#40431)

Commit migrated from https://github.com/dotnet/corefx/commit/41cd99d051102be4ed83f4f9105ae9e73aa48b7c

Fix official builds because of SDL authentication (dotnet/corefx#40483)

* Fix official builds because of SDL authentication

Commit migrated from https://github.com/dotnet/corefx/commit/a1578465f68f6793e7498152dc2d0d2f4eed3a4e

Fix namespace typo in test (dotnet/corefx#40460)

Commit migrated from https://github.com/dotnet/corefx/commit/6257672b2172bf993568cf57ff5d2397bf1dcc7d

Fixed filename case issue (dotnet/corefx#40457)

Commit migrated from https://github.com/dotnet/corefx/commit/c71d095e3e57e0ad2a75d8d97b93cdceb4f7f720

FromBase64Transform.InputBlockSize set to 4 instead of 1

Commit migrated from https://github.com/dotnet/corefx/commit/b456415bca6317b1ad6f98c6475ca59a20c57101

Update Debug.Assert to more clearly show the condition (brace for objects, bracket for array).

Commit migrated from https://github.com/dotnet/corefx/commit/30e06a4f4359f9a368e8fbab013259f46c866df4

Avoid WeakReference race with GC in LdapSessionOptions (dotnet/corefx#40340)

The GC could run between the check of IsAlive and the access to Target.  Better to just access Target and use the result if it's non-null.

Commit migrated from https://github.com/dotnet/corefx/commit/2fac89f63c644ab81d936d62c91e6003302e718d

Remove dead code in CacheMemoryMonitor.SetLimit (dotnet/corefx#40336)

The code was initializing _memoryLimit to 0.  That then makes the second branch of the cascading if/else if/else if dead and the third branch into a pure else.  As a result the whole structure can be brought down to a simple ternary.

Commit migrated from https://github.com/dotnet/corefx/commit/103fc3a785dc8d927357cb3614468fd31ef98864

Use UTC date in Json{Object,String}Tests which use MinValue

Commit migrated from https://github.com/dotnet/corefx/commit/4f7dc2e9d56035d514caa92b9a61d5c03f28c3d8

Fix reading past comments after an end object or end array token within Utf8JsonReader (dotnet/corefx#40436)

* Fix reading past comments after an end object or end array token.

* Add more test cases and update unreachable code paths with
Debug.Asserts.

Commit migrated from https://github.com/dotnet/corefx/commit/247793c64816a887c4ff47a92acdd9b527b7873b

Fix BitArray.CopyTo to byte[] regression - update offset for extra bits. (dotnet/corefx#40441)

Commit migrated from https://github.com/dotnet/corefx/commit/2547b52f31131f10b3cefc530242df35ce9d0e70

Fix failing System.Text.Json test (dotnet/corefx#40439)

Commit migrated from https://github.com/dotnet/corefx/commit/cf5c1b0a7c3cf0087ae72fd0e1995a274cfefe27

System.Text.Json - Don't add to readonly collections - Take 2 (dotnet/corefx#40366)

* Don't add to collection when IsReadOnly is true

* Throw NotSupportedException instead of JsonException

* Add test for readonly collections

* Revert back to InvalidCastException

Commit migrated from https://github.com/dotnet/corefx/commit/f66beeec1ab37a8ea8647caae8bb2ae901f01012

Fix Outerloop runs

Commit migrated from https://github.com/dotnet/corefx/commit/97abb7eb388206f0f976440bbde8f652e9cdd5e1

Use named argument syntax for null values in json serialization methods (dotnet/corefx#40435)

Commit migrated from https://github.com/dotnet/corefx/commit/2b2dd5120029081af767e0eea5e813fd68bded11

Make sure byte position for invalid JSON is consistent and the exception message is accurate (dotnet/corefx#40349)

* Add tests for reading numbers when data is within a multi-segment
ReadOnlySequence.

* Test rename and add debug.assert.

* Test clean up - remove unnecessary/expensive Debug.Assert.

* Test and code cleanup.

* Address feedback - add stronger test condition for token length.

* Revert state on incomplete read and on error. Make sure exception
message is accurate.

* Fix byte position in line for invalid JSON and update tests.

* Address feedback - pass by in and move private struct to end.

* Fix string consumption and add various tests.

* Fix off-by-one error when calculating position.

* Test clean up - make sure multi-segment tests are in the right file.

* Update string parsing - do not increment line number for escaped newline
chars within quoted strings.

* Remove unnecessary custom state struct for string and re-use the same
one for both string and numbers.

* Update tests based on PR feedback.

* Create helper to get start sequence position from roll back state.

Commit migrated from https://github.com/dotnet/corefx/commit/a690f1658cdc4d92936bd709e644074b2e2e288e

Re-enable xunit warning 1019 (dotnet/corefx#40415)

Make MemberData reference IEnumerable<object[]>

Commit migrated from https://github.com/dotnet/corefx/commit/3c09762fa14249c65e5bc676a5d812c2654c8941

Honor converters for implementing types (dotnet/corefx#40411)

* Honor converters for implementing types

* Cache implementing-type converter if one is detected

* Address feedback: name parameters and add IEnumerable wrapper test

Commit migrated from https://github.com/dotnet/corefx/commit/ee880076f3bcacde1a32e88ea4ebd701a609e52d

Use new instances of JsonSerializerOptions in property visibility tests to prevent re-using cached metadata (dotnet/corefx#40414)

Commit migrated from https://github.com/dotnet/corefx/commit/21ce6673922a93d34117b6c757f394600b7f7dbb

Add DecoratedNameAttribute (dotnet/corefx#40125)

Commit migrated from https://github.com/dotnet/corefx/commit/86db0ef4acc409ab29cd07908aec21c2dc237ea0

Honor JsonIgnore attribute when applied to unsupported collections (dotnet/corefx#40401)

* Honor JsonIgnore attribute when applied to unsupported collections

* Address feedback: add more unsupported collections

Commit migrated from https://github.com/dotnet/corefx/commit/6739eab2f4ee9a895575289893edf63b6c234d43

Avoid using double/floats within tests that don't roundtrip on netfx (dotnet/corefx#40408)

(for G17).

Commit migrated from https://github.com/dotnet/corefx/commit/b2ea249aa67d719b35c6a1c95befbf2b819bab17

Corefx dotnet/corefx#40133 Use TryFormatDouble TryFormatFloat (dotnet/corefx#40197)

* Corefx dotnet/corefx#40133 Use TryFormatDouble TryFormatFloat

* add test cases

* Changes as per review comments

* remove extra empty line

Commit migrated from https://github.com/dotnet/corefx/commit/3d72b56b8e7d1e4e44262be6c45c2aac580bbbc4

Change 'type' to 'inputType' on serialize methods (dotnet/corefx#40372)

Commit migrated from https://github.com/dotnet/corefx/commit/9217d338671fe78d2bd31138a610bb9ad5a12608

Use Count in Enumerable.Any if available (dotnet/corefx#40377)

* Use Count in Enumerable.Any if available

We've been hesitant to make this change in the past, as it adds several interface checks.  However, wide-spread "wisdom" is that `Any()` is as fast or faster than `Count() > 0`, and there are even FxCop rules/analyzers that warn about using the latter instead of the former, but in its current form that can frequently be incorrect: if the source does implement `ICollection<T>`, generally its `Count` is O(1) and allocation-free, whereas `Any()` will almost always end up allocating an enumerator.  On balance, it seems better to just have `Any()` map closely to `Count()` so that their performance can be reasoned about in parallel.

* Add test coverage for Enumerable.Any

Commit migrated from https://github.com/dotnet/corefx/commit/9021bc1cbc8c1189d2eb8de906e8891878430e6f

Disable AssociatePersistedKey_CAPIviaCNG_RSA on arm64 (dotnet/corefx#40375)

Commit migrated from https://github.com/dotnet/corefx/commit/9ea74a6d3ec09fef63975338105abdca6fac8e1c

Remove unnecessary field in PrinterSettings (dotnet/corefx#40339)

I'm not sure why it has its own ArrayEnumerator, but as long as it does, it doesn't need _startIndex.

Commit migrated from https://github.com/dotnet/corefx/commit/bfe2c58a4536db9a257940277c5d94bf9e26929a

Disabling test failing in Mac OSX 10.15 (dotnet/corefx#40332)

* Disabling test failing in Mac OSX 10.15

* Using active issue annotation

* Checking OS version instead of disabling test for any Mac

* Formatting update

Commit migrated from https://github.com/dotnet/corefx/commit/df3837cbced3fdcaab22464c18f0ce20b4aadfc7

Fix outerloop cron job (dotnet/corefx#40317)

Commit migrated from https://github.com/dotnet/corefx/commit/001f42b944e3a05a42f9f8cd7d14d338a06e6328

Remove unnecessary null check from ConstraintCollection (dotnet/corefx#40343)

`_table` will never be null.  The `ConstraintCollection` is only constructed in one place, where it's passed `this` by `DataTable`.  It's used in many other places without a guard check, as well (and later in this method, unguarded).

Commit migrated from https://github.com/dotnet/corefx/commit/f4344b0d3b45fc7c8f41e5c909ddcb2752bea0fa

Fix IDE0066 across corefx (dotnet/corefx#40288)

* Fix IDE0066 across corefx

"Fix all", but then manual tweaking to accommodate places the rule doesn't work quite right.

* Address PR feedback

Commit migrated from https://github.com/dotnet/corefx/commit/f71a5a1d88214a9e2c26ff2ff8bcfb1017aa6713

Improve code coverage for VB Conversion module (dotnet/corefx#39937)

Commit migrated from https://github.com/dotnet/corefx/commit/4d2b68c2cda66c861aa918fd91e9bf7d664ea05f

Fix BytesConsumed when reading JSON numbers within a multi-segment ReadOnlySequence via Utf8JsonReader (dotnet/corefx#40303)

* Add tests for reading numbers when data is within a multi-segment
ReadOnlySequence.

* Test rename and add debug.assert.

* Test clean up - remove unnecessary/expensive Debug.Assert.

* Test and code cleanup.

* Address feedback - add stronger test condition for token length.

Commit migrated from https://github.com/dotnet/corefx/commit/1b0f28942ccafca8a56f960536e125301e7b9306

Minor cleanup tweaks in Regex (dotnet/corefx#40338)

* Remove unused local in RegexRunner.TextposDescription

* Remove redundant checks in RegexCharClass.AddRange

Commit migrated from https://github.com/dotnet/corefx/commit/04bd3fa5c2c7d379c22b805ad840bc28423d5242

Remove duplicate _wrapped.WriteBase64 call in QueryOutputWriterV1 (dotnet/corefx#40345)

The same expression is used for both the if and the else branches.  But the if branch does make a mutating method call, which we keep.

Commit migrated from https://github.com/dotnet/corefx/commit/6aaba8d6f07c8f2e4db0a918dbdbad6d95b26eae

adding ci to release 3.0 branch (dotnet/corefx#40359)

Commit migrated from https://github.com/dotnet/corefx/commit/c5cb132f5ad2b938e67866cf510d3f3bc58e32c3

Avoid creating/populating unneeded certificate collection in SecureChannel.AcquireClientCredentials (dotnet/corefx#40354)

Commit migrated from https://github.com/dotnet/corefx/commit/f34e088ff8400ff39ceff5a68b3b1a90dbe8d0ab

Tests for JsonSerializerOptions.GetConverter() and converter methods (dotnet/corefx#39750)

* Tests for JsonSerializerOptions.GetConverter

Commit migrated from https://github.com/dotnet/corefx/commit/e11af6abaaec093cd77cb55738b99b466f70148e

Support custom converters that treat non-null input as null (dotnet/corefx#40287)

Commit migrated from https://github.com/dotnet/corefx/commit/ba54c4034a0e87ac6bdf3d13b55b767f5c321df2

Prevent threading issues with warm-up of property caching (dotnet/corefx#40261)

Commit migrated from https://github.com/dotnet/corefx/commit/b35cfc014b17cca8ee20c24ee250c6925d73855f

Renaming Microsoft.Framework.WebEncoders namespace (dotnet/corefx#40238)

* Renaming the namespace Microsoft.Framework.WebEncoders to System.Text.Encodings.Web.Tests

- Namespace Microsoft.Framework.WebEncoders renamed to System.Text.Encodings.Web.Tests

* Changed aliases naming from camelCase to PascalCase

* Deleted unnecessary using directives and reverted EncoderExtensionsTests

Deleted unnecessary using directives. Reverted changes made in EncoderCommonTests.cs to commit dotnet/corefx@db1dd29edac07716ac27b8916d5e12f9a85b4978 and changed using directive for test namespace to right one.

Commit migrated from https://github.com/dotnet/corefx/commit/1a8fab5315d2de425d530c6fb7d801ade653bc41

fixing ZipPackagePart.GetStreamCore crashes with NotSupportedException (dotnet/corefx#40319)

* fixing ZipPackagePart.GetStreamCore crashes with NotSupportedException

ZipArchiveEntry only ever supports opening once when the backing archive is in Create mode,  and the backing stream is non-seekable, so we shouldn't call SetLength in that case. You could still open an archive in Update mode then call part.GetStream(FileMode.Create), in which case we'll want this call to SetLength, so we only avoid this call when the backing Archive is in Create mode.

* updating test to explicitly test the Update path for ZipPackage

* skip UAP since we don't have access to the file system to create the .zip

* undo accidental change to existing test

* removing unnecessary variable

Commit migrated from https://github.com/dotnet/corefx/commit/8ac53e0c022b7881f659e0838a25e91f2ec5c8c0

Use sealed securityContext in SecureChannel (dotnet/corefx#40265)

* Use sealed securityContext in SecureChannel

* Less asm

* Feedback

* Remove ternary

Commit migrated from https://github.com/dotnet/corefx/commit/6e53bf857a5cbfa1a6b26be3173e00b7e032e819

Remove duplicate code in WMIGenerator.ConvertToNumericValueAndAddToArray (dotnet/corefx#40346)

Commit migrated from https://github.com/dotnet/corefx/commit/203b1fda97a7421b24a0a41ef9459e0fc31efe3d

Apply feedback to pal_x509chain.c fix from release/3.0

Commit migrated from https://github.com/dotnet/corefx/commit/cc1f32affb828e1a34a4c91d8e7ba1474fe196b3

Fix a few missing $s for string interpolation in tracing (dotnet/corefx#40322)

Commit migrated from https://github.com/dotnet/corefx/commit/d116b2d4ee4041970943d6fffdf8a97446f8b6ff

Fix non-ASCII characters across corefx (dotnet/corefx#40318)

* Fix several non-visible characters in test files

* Replace Unicode surrogate pairs with escaped hex values

* Fix non-ASCII characters in most C# files

Commit migrated from https://github.com/dotnet/corefx/commit/027eb8e19c3498cc2c656931c3b879f5ddb63b0d

Delete dead code from XMLDiffLoader.ReadOldRowData (dotnet/corefx#40344)

We already exited out of the method earlier if table == null.

Commit migrated from https://github.com/dotnet/corefx/commit/323b0c158e3d6ece5a8cb1c87c443848cbde9afd

Fix PKCS1MaskGenerationMethod.GenerateMask handling of invalid hash name (dotnet/corefx#40323)

* Fix PKCS1MaskGenerationMethod.GenerateMask handling of invalid hash name

* Address PR feedback

Commit migrated from https://github.com/dotnet/corefx/commit/73b8056a88278b31752fdfbafbfe7479a0540767

Remove unnecessary null check from QueryDynamicObject (dotnet/corefx#40320)

Commit migrated from https://github.com/dotnet/corefx/commit/9982320d1d1185e30507152687ecd755ea95bb2b

Remove dead else branch from SchemaGraph.Depends (dotnet/corefx#40325)

Commit migrated from https://github.com/dotnet/corefx/commit/618de6e29d0c5b8b4cab65bdb98e6893240ab77d

Remove duplicate casts from DbMetaDataFactory (dotnet/corefx#40328)

Commit migrated from https://github.com/dotnet/corefx/commit/66273ed9944c7c964d3d51771ed8d4b4ddce3e8c

Remove redundant code in DirectorySearcher (dotnet/corefx#40321)

Commit migrated from https://github.com/dotnet/corefx/commit/f7ecc218455c41bb4c5c13b9d277a22a79146756

Remove duplicate branch in ColorTranslator.ToHtml (dotnet/corefx#40337)

Commit migrated from https://github.com/dotnet/corefx/commit/87d6aa5daf9b5b7a01174c3849c731ecfbee3f29

Avoid tracing-induced null ref in HttpListener when {Begin}Read/Write called with invalid buffer (dotnet/corefx#40335)

HttpRequest/ResponseStream's {Begin}Read/Write methods trace the length of the caller-supplied buffer.  If the caller erroneously passes null, with tracing enabled it'll end up throwing a NullReferenceException instead of an ArgumentNullException.  This should never happen in real-world code, but might as well clean it up.

Commit migrated from https://github.com/dotnet/corefx/commit/04f9d7b23fd9e9c4d8499436f1a71742b5f640ee

Fix null ref in ImmutableArray's structural comparer (dotnet/corefx#40341)

Commit migrated from https://github.com/dotnet/corefx/commit/a87fcffbd1606e4f29c8f0473cde2ae3f2209347

Delete redundant Debug.Assert in XPathNodePointer.ComparePosition

Commit migrated from https://github.com/dotnet/corefx/commit/ed56010a736dfc242edb07bf3c501696aad840fc

Remove unnecessary null check from JsonClassInfo.CreatePolymorphicProperty (dotnet/corefx#40334)

`property` will never be null in calls to this method, and even if it were, prior dereferences of it would result in exceptions.

Commit migrated from https://github.com/dotnet/corefx/commit/324e6432b99c3c8a655d7db2de57811b12967450

Remove stale ctor and field from ObjectManager (dotnet/corefx#40326)

Commit migrated from https://github.com/dotnet/corefx/commit/bfe528311e52fd41afca1c981dc28c2e62d96721

Remove dead branches from ObjectManager (dotnet/corefx#40327)

int.MaxValue is odd.  x * 2 will never be odd.

Commit migrated from https://github.com/dotnet/corefx/commit/c121f180b985f3bb2ca28096369eda7b7255af7b

Fix potential null ref in RuntimeInformation.FrameworkDescription (dotnet/corefx#40329)

If there's no FX_PRODUCT_VERSION and then if there's no AssemblyInformationalVersionAttribute on the assembly or its InformationVersion is null, we'll try to dereference a null string.

Commit migrated from https://github.com/dotnet/corefx/commit/c08abbe3527fca16b55df53a596164fac97031d4

Add comment to explain base.Pop call in CorrelationManager

Commit migrated from https://github.com/dotnet/corefx/commit/36f1a74b4ecf04f2453c0e23265bb21500bbd45d

JsonObject implementation and tests (dotnet/corefx#40206)

* JsonObject implementation added
* JsonObject tests added
* JsonObject documentation added
* Review comments included

Commit migrated from https://github.com/dotnet/corefx/commit/ceaad9274f83273a4c95ec08b359253ad28bea1e

Add IPEndPoint tests and fix some bugs (dotnet/corefx#39726)

* Cleanup IPEndPoint

* Add missing validation to IPEndPoint

Commit migrated from https://github.com/dotnet/corefx/commit/7224d186f90e28f0c04eefb875260c03d6acec07

Fix JSON Writer docs for data writter as Base64-encoded text (dotnet/corefx#40200)

* Fix JSON Writer docs for data writter as Base64-encoded text

* PR Feedback

* More Feedback

* One more try

Co-Authored-By: Ahson Khan <ahkha@microsoft.com>
* Use Base64 instead

* As a to as for text

Commit migrated from https://github.com/dotnet/corefx/commit/39d8cf14b1eaf2d7678d1399bc903a0e547cce26

Add NotNullIfNotNull attribute to Interlocked.CompareExchange (dotnet/corefx#40258)

Commit migrated from https://github.com/dotnet/corefx/commit/b990ece3f8cd7999ad62483f1bd814ec586895de

Make many fields readonly or const across corefx (dotnet/corefx#40298)

* Make many fields readonly or const

* Address PR feedback

- Updated asn.xslt
- Regenerated .xml.cs files

Commit migrated from https://github.com/dotnet/corefx/commit/0cc22ccba707d7552159363d586262fa9c4e8a2a

Consolidate the Image class, fix bug (dotnet/corefx#40181)

* Add new values for Encoder, EncoderParameterValueType

* Support encoder parameters of type ValueTypePointer

* Fix an overflow error when reading EncoderParameters

* Add unit tests

* Consolidate the Image class

Share the following methods across Windows & Unix:
- FromFile
- GetEncoderParameterList
- GetPixelFormatSize
- IsAlphaPixelFormat

* Fix P/Invoke declaration

* Fix compliation on NetFX

* Skip test on NetFX

* Fix typo

* Always skip GetEncoderParameterList_ReturnsExpected on netFX

* Remove new API

* Revert changes which require an API change

* Suppress failing test

* Image: Consolidate CreateImageObject
Metafile: Consolidate initialization from native handle

* Throw SkipTestException instead of silently skipping the test

* Enable some on Unix which rely on consolidated code

* Add missing using statement

* Get SkipTestException right

Commit migrated from https://github.com/dotnet/corefx/commit/31a04af352c0046b21b2ccdf5b409203a43fae1f

Fixed Dispose in JsonDocument and JsonDocument.MetadataDb in order to guard against multiple calls to ArrayPool.Return (dotnet/corefx#40290)

* Fixed Dispose in JsonDocument and JsonDocument.MetadataDb in order to guard against multiple calls to ArrayPool.Return.

* Added condition to validate _utf8Json.Length is > 0 in order to prevent that extraRentedBytes.AsSpan(0, length).Clear() takes a zero length range in a race condition.

* Added test related to this scenario.

* Addressing suggestions.

* Removing unnecesary null assingment.

* Adding comment explaining the reason for minimumLength = 4 on ArrayPool.Rent.

* Adding using System.Threading.

Commit migrated from https://github.com/dotnet/corefx/commit/f47bb86d84c5e08e1a8520c5f2b23a0bd7652e2a

[master] Update dependencies from dotnet/coreclr (dotnet/corefx#40309)

* Update dependencies from https://github.com/dotnet/coreclr build 20190813.7

- Microsoft.NET.Sdk.IL - 5.0.0-alpha1.19413.7
- Microsoft.NETCore.ILAsm - 5.0.0-alpha1.19413.7
- Microsoft.NETCore.Runtime.CoreCLR - 5.0.0-alpha1.19413.7

* Fix attribute target in ref

Commit migrated from https://github.com/dotnet/corefx/commit/4c4dc052c675394db5ca78f0503b350976599202

Remove Microsoft.VisualBasic.{ApplicationServices,Devices,MyServices} (dotnet/corefx#39972) (dotnet/corefx#40292)

Commit migrated from https://github.com/dotnet/corefx/commit/02c729c4b6b3c705b9e19d1c819b17128a85c01a

Enable Sdl Validation (dotnet/corefx#40294)

Commit migrated from https://github.com/dotnet/corefx/commit/df6c87128e0407b21103fe00680f498787631d5a

Fix problematic characters in XmlJsonReader.cs (dotnet/corefx#40291)

A few stray invisible characters found their way into this .cs file and appear to be causing Visual Studio to be very unhappy.

Commit migrated from https://github.com/dotnet/corefx/commit/f0ec4a345fc31bc9d075fec86fa6fb3ec88d43a2

Add TLS 1.3 to all protocols in GetAsync_AllowedSSLVersion_Succeeds (dotnet/corefx#40281)

Commit migrated from https://github.com/dotnet/corefx/commit/302b3283f5439907ca2586784ba32381ac312f72

Use Strings.resx file within System.Text.Encodings.Web (dotnet/corefx#40239)

* Use Strings.resx file for the exception messages within System.Text.Encodings.Web

* Add reference to System.Resources.ResourceManager

* Change exception message

Commit migrated from https://github.com/dotnet/corefx/commit/6f8983d64c67fa86ac6f0338cde08d82e51cc805

Make serialization null tests independent of reflection ordering (dotnet/corefx#40257)

Commit migrated from https://github.com/dotnet/corefx/commit/ab78eb491689e1b6694e4543b74d2fa8a158fd08

Performance improvements in the CryptoConfig class

CreateFromName is about 10x faster with this change.

* Use ConcurrentDictionary instead of big locks.
* Cache results of DefaultNameHT lookups resulting in string-to-Type into AppNameHT.
* Specify initial capacity when creating DefaultOidHT and DefaultNameHT dictionaries.

Commit migrated from https://github.com/dotnet/corefx/commit/963e0d64bd7a69ba4b8d4d933bd56d0f2a5a8913

Fix 2 System.Net.Http tests failing on debian (dotnet/corefx#40259)

Commit migrated from https://github.com/dotnet/corefx/commit/af36939acdf751d2465358be3f892170f73f9b48

Avoid disposing Http2Stream's request body cancellation source (dotnet/corefx#40255)

Commit migrated from https://github.com/dotnet/corefx/commit/a4f5f13ec1fa0d51ce0964c126e35ef0046ea7be

SqlClient: Fix BulkCopy double and single null widening handling (dotnet/corefx#37603)

* change BulkCopy double and single handling to handle Null values when converting to decimal without throwing and add tests

* address feedback

Commit migrated from https://github.com/dotnet/corefx/commit/97ddad9e75b241e752242c4e33c3e68f7fe0d5f3

Enable StyleCop rules 1205, 1411, 1026, and 1000 (dotnet/corefx#40254)

* Fix StyleCop warning SA1411 (attribute ctor shouldn't use unnecessary parens)

* Fix StyleCop warning SA1026 (new[] spacing)

* Fix StyleCop warning SA1000 (consistent spacing around if, for, switch, etc.)

* Fix StyleCop warning SA1205 (partial elements should declare an access modifier)

Commit migrated from https://github.com/dotnet/corefx/commit/0c4c234a918254d569ffb0f6a607f98193608646

Make several local functions static (dotnet/corefx#40251)

Commit migrated from https://github.com/dotnet/corefx/commit/d3155017e1fe6acdb72e2d1b30c5b2e36cf21af8

Disable two SocketsHttpHandler tests on Debian10 (dotnet/corefx#40247)

Commit migrated from https://github.com/dotnet/corefx/commit/eb58b0a0e0a45c42d67b69b44047cf60116650c3

Re-enable scheduled outerloop runs (dotnet/corefx#40244)

Commit migrated from https://github.com/dotnet/corefx/commit/103456e6b0d122a4b1952e7b6dc38642f6eed2db

Disable FinancialTest.FV test on arm64 (dotnet/corefx#40246)

Commit migrated from https://github.com/dotnet/corefx/commit/e446e9e774fce79e50fc29c5f57a2861e47fd6a3

Re-enable StyleCop warning SA1400 (elements should specify access) (dotnet/corefx#40240)

One of the more common things we "nit" in PRs.  Now the compiler will flag it for us.

Commit migrated from https://github.com/dotnet/corefx/commit/61a6264018989e06d314b68d57bd91b4b72e4360

Disable System.Drawing tests failing on arm64 (dotnet/corefx#40223)

Commit migrated from https://github.com/dotnet/corefx/commit/d3d48eb94c38d22725fc348b445a80136979be17

Upgrade Windows Nano Server to 1809 (dotnet/corefx#39734)

* Upgrade Windows Nano Server to 1809

* Handle ERROR_CALL_NOT_IMPLEMENTED in PlatformDetection.IsInAppContainer

* Disable TextInfo test on Nano server

* Disable EnvironmentVariable tests on user scope

Commit migrated from https://github.com/dotnet/corefx/commit/61c5e96b70fd14621ec71aee356b1b520696a3c9

Disable two EventLog failing tests on Windows (dotnet/corefx#40225)

Commit migrated from https://github.com/dotnet/corefx/commit/8f86714ab411924ba83dcf0928130a3437eeba88

followup on SocketAddress validation (dotnet/corefx#40165)

* followup on SocketAddress validation
* update unknow faimily tests
* feedback from review
* fix spelling
* add comments about new values
* feedback from review

Commit migrated from https://github.com/dotnet/corefx/commit/b129f7657a1b93ce9cf577d769d9d03c862e2338

Fix bugs in null serialization (dotnet/corefx#40077)

* Support null serialization for objects and dictionaries as values in Dictionary and KeyValuePair

* Address reviw comments

* Fix nested enumerable null serialization and add dictionaries to null test suite

Commit migrated from https://github.com/dotnet/corefx/commit/4fbef813c6e3c1ce1a23e5dc1528e65abb1b8c0a