Merge pull request #4540 from kumpera/android-changes-part1

[bcl+runtime] Multiple small cleanups to Android.

[tools] Remove unused sources

[jit] Cleanup the signature of the mono_arch_get_static_rgctx_trampoline () function. (#4605)

Remove the unused MonoMethod argument, and rename the 'mrgctx' argument to 'arg', since it can be any value, not just
an mrgctx.

Bump cecil

Bump corefx

[jit] Add ref types to mini_get_underlying_type (), use it to simplify the switch loops in the jit. (#4606)

[threadpool] Pass callback at initialization (#4546)

This remove the possibility to execute different callbacks on the threadpool, but this allows us to remove the work_items_lock, which is called on the hot path of the ThreadPool.

[msvc] Update csproj files

Merge pull request #4168 from xmcclure/aot-man

Correctly document --aot options in man page

Remove 'experimental' label from full/hybrid AOT in manpage

Correctly document --aot options in man page

- Ensure all AOT options are both documented and listed in
  alphabetical order in both the manpage and --help line
- Ensure all exceptoins are commented in the aot-compiler.c source
  with "intentionally udnocumented"

[mono] Remove unused icall

Update copyright years

[corlib] Add back X509 support for watchOS.

Otherwise we'd be regressing existing (and working) behavior.

Disable a few x509 tests that fail on older macOS

- Handles https://bugzilla.xamarin.com/show_bug.cgi?id=53958
- https://bugzilla.xamarin.com/show_bug.cgi?id=53689 is filed to turn them
  back on later with a better version check
- SecCertificateCreateWithData handles invalid arguments differently based on
  macOS version

[corlib] Add more Array tests

[System.Windows.Forms] Fixes DataGridViewColumnCollection comparer not to return 1 on match

[System.Windows.Forms] Imrove DataGridViewColumnCollectionTest

[corlib] Use array managed parts from corert to fix bugs in sorting and arguments checking. Fixes #53196 and similar

Bump corert

[corlib] Fix RegistryKeyTest cleanup

Merge pull request #4594 from lambdageek/bug-53890

[remoting] Pass correct object to invoke of IRemotingTypeInfo.CanCastTo

Merge pull request #4570 from lateralusX/jlorenss/visual_studio_msbuild_fix

Windows msbuild triggers too much to rebuild.

Add FieldInfoMirror.FullName (#4588)

[interp] call mono_unhandled_exception, so exit() is called with proper return code

[interp] enable more tests for CI

[interp] support floating point in runtime_invoke

[interp] allow more gregs, enable pinvoke17.exe

[interp] throw NRE in mint_strlen

[interp] add write barriers

issues found with
`MONO_GC_DEBUG=3,collect-before-allocs,check-remset-consistency ../mini/mono-sgen --interpreter /Users/bernhardu/work/mono/b/lib/mono/4.5/mcs.exe hello.cs`

[interp] respect reference type in stobj insn

stackval_from_data does everything for us.

[remoting] Pass correct object to invoke of IRemotingTypeInfo.CanCastTo

Pass the payload of the coop handle as the 'this' object, not the handle
itself.

Add asserts in ves_icall_Remoting_RealProxy_InternalGetProxyType that
remote_class and the proxy_class are non-NULL.

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=53890

[docs] Update formatting in mono-api-utils.

[docs] Update formatting in mono-api-unsorted.

[docs] Update formatting in mono-api-type.

[docs] Update formatting in mono-api-threads.

[docs] Update formatting in mono-api-string.

[docs] Update formatting in mono-api-security.

[docs] Update formatting in mono-api-reflection.

[docs] Update formatting in mono-api-profiler.

[docs] Update formatting in mono-api-object.

[docs] Update formatting in mono-api-methods.

[docs] Update formatting in mono-api-metadata.

[docs] Update formatting in mono-api-jitinternal.

[docs] Update formatting in mono-api-jit.

[docs] Update formatting in mono-api-internal.

[docs] Update formatting in mono-api-image.

[docs] Update formatting in mono-api-gchandle.

[docs] Update formatting in mono-api-gc.

[docs] Update formatting in mono-api-exc.

[docs] Update formatting in mono-api-embedding.

[docs] Update formatting in mono-api-dynamic-codegen.

[docs] Update formatting in mono-api-domains.

[docs] Update formatting in mono-api-debug.

[docs] Update formatting in mono-api-counters.

[docs] Update formatting in mono-api-class.

[docs] Update formatting in mono-api-class.

[docs] Update formatting in mono-api-assembly.

[exdoc] Enable formatting in deprecated annotations.

[exdoc] Handle punctuation better in code formatting.

[docs] Use Doxygen syntax for Monodoc.

[exdoc] Update warnings.

[exdoc] Support Doxygen syntax.

[docs] Fix exdoc warnings.

[exdoc] Refactor.

Avoid globals, use proper data structures instead of strings, and
produce some warnings for documentation errors.

[exdoc] Add missing dependency.

[exdoc] Enable 'warnings' and 'strict'.

[exdoc] Fix whitespace.

[docs] Remove redundant filename.

[docs] Fix Doxygen errors.

Doxygen treats '@code' as the start of a '@code'...'@endcode' block, so
it raised spurious "reached end of file while inside code block" errors
when '@code' was used to denote a parameter. Change these instances to
'\param code' (declaration) and '\p code' (mention).

[docs] Enable documentation for utils.

[docs] Enable documentation for mini.

[docs] Enable documentation for sgen.

[docs] Enable documentation for metadata.

[docs] Improve Doxygen config.

Revert "Update corefx"

This reverts commit 2a2c317d07e389088737b391e97853fdb6dbc390.

[tests] Remove duplicate property

Merge pull request #4590 from xmcclure/master

Improve "Could not resolve type with token" exception message

Improve "Could not resolve type with token" exception message

Update corefx

[mcs] Reset also all partial parts current-type

Merge pull request #4503 from BrzVlad/fix-appdomain-unload

[runtime] Fix abort issues

A few fixes to the man page

[bcl] Add a finalizer to MemoryMappedFile.

[System.Data] Update tests for mobile profiles

[runtime] Don't reset abort exception in invoke wrapper

It leads to missing self aborts on full-aot.

[runtime] Improve debugging of thread interruptions

[runtime] Defer async requests during abort protected blocks

We avoid unnecessary checks for each interruption checkpoint.

[runtime] Fix detecting abort at end of abort protected block

If we have a pending abort then we know we will have the thread_state set and the thread_interruption_requested bumped. We used to consume this request and set the MonoError with the exception. The problems with this is that not all callsites handle the passed MonoError and handling it means converting it back to a pending exception anyway. We just leave the interruption request untouched now.

If the cctor self aborts it means that we consumed the abort exception so we do need to call mono_thread_resume_interruption to reactivate it. We need to make sure that we still don't throw a TypeInitializationException if we had a self abort.

[AppleTls]: Use correct definition of `SecTrustResult`. (#4586)

* [AppleTls]: Use correct definition of `SecTrustResult`.

We need all the values in this enum because `SecTrustResult.Unspecified`
means success - which currently has an integer value of 4, not 0.

* Oops, make it build.

* Don't make it public.

* Revert whitespace change.

Bump version to 5.1.0

Add test for AutomaticDecompression behaviour

Test that HttpWebRequest.AutomaticDecompression actually performs the
decompression and that it removes the Content-Encoding header. Also fix
the value of Content-Encoding for other tests to follow the HTTP
specification.

Remove Content-Encoding header after doing automatic decompression

Match the .NET Framework behaviour by removing the Content-Encoding
header after doing automatic decompression.

[msvc] Update csproj files

[reflection] Set correct reflected type in MonoMethod.get_base_method

It can be the case that we have two different System.MonoMethod objects that both
point to the same native MonoMethod* but have two different reflected types.
(For example if we call derivedType.GetMethod("f") where f is virtual and
derivedType corresponds to a class that did not override a base definition.  In
that case, the MonoMethod* corresponds to the parent type's f method, but the
reflected type is derivedType)

In this case, GetBaseDefinition will return the same MonoMethod* but the
reflected type of the result should be the base type, not the derived type.

[tests] Additional tests of System.Reflection.MethodInfo.GetBaseDefinition

[threadpool-io] Ensure selector thread is running before waiting on it (#4572)

This could result in a unecessary wait on shutdown where the selector thread would have exited, and the finalizer thread would try to dispose a Socket, which would try to remove the socket from the IOSelector. This removal operation would wait on the selector thread to acknowledge the removal, but because the selector thread would have alreday exited, we would wait on an event that would never happen.

[mkbundle] Use correctly link paths when loading assembly deps

invoke_array_extract_argument() will create objects and set them in the parameters array when it detects a null parameter, but when we return a by-value parameter may no longer be null, which is an error.  This fix detects the change and sets the parameter back to null for by-value parameters before we return.

One brace missed