[System.Net.Http] Simplify default http message handler for watchOS since there's only one valid value.

[ci] Temporarily switch back to legacy TLS provider

[Mono.Debugger.Soft] Tweak ClassLocalReflection test

Merge pull request #3691 from ntherning/enable-tests-in-pr-builds-on-windows

Enable tests in Windows PR builds

[System*] Throw a PlatformNotSupported exception when using the managed networking stack on watchOS. (#3683)

* [System.Data] System.Data.SqlClient is not supported in watchOS.

System.Data.SqlClient is not supported in watchOS, because it depends on a
working managed networking stack, which we don't have on watchOS.

* [System] Throw PlatformNotSupportedExceptions in the networking stack for watchOS.

* [System] Don't reference/build Mono.Security.dll when building for watchOS.

This means we can't build Mono.Data.Tds.dll either, since it uses
Mono.Security.dll extensively. In any case Mono.Data.Tds.dll is useless
without a working networking stack, and since it's a Mono-only dll (i.e. not
part of netstandard or facades), just remove the entire assembly from the
watchOS profile.

System.Security.dll uses Mono.Security.dll as well, but the required classes
are available inside mscorlib.dll (as internal classes), so just open up
mscorlib internals to System.Security.dll.

* [System] Update tests after API change for watchOS.

* [System.Data] Update tests after API change in watchOS.

* Revert "Fix build break from 81fb20737f810def6dde88d87a1b2f23cdb736c3"

This reverts commit f4475ff01358f8490e7cd0878e8b39bed2a0def7.

* Fix missing trailing newline in monotouch_watch_System.dll.sources

It breaks the build on Linux if the newline is missing.

Enable tests in Windows PR builds

Also run the test suite in Windows PR builds. We disable all the test suites
in Windows PR builds that are known to fail.

[System] Server property of HttpWebResponse should never be null

[btls] Fix i386 build on Linux by setting -arch on Darwin only

The -arch option is an Apple extension, it caused failures on i386 Linux: "unrecognized command line option '-arch'"

Merge pull request #3689 from alexrp/profiler-c9-fixes

Profiler fixes for Cycle 9 (#2)

[profiler] Fix GC finalize object end event not getting invoked.

Revert "[domain] Report domain unloads to the profiler when finalizing the domain."

This reverts commit c8b20290808563f89d29b22d2fe35eb906cfe725.

This caused ordering problems with app domain events: https://bugzilla.xamarin.com/show_bug.cgi?id=44552

Conflicts:

mono/metadata/gc.c

Merge pull request #3688 from alexrp/profiler-c9-fixes

Profiler fixes for Cycle 9

[profiler] Bump profiler version to v1.1.

[profiler] Don't emit enter/leave events in onlycoverage mode.

[profiler] Add a nocounters option.

[profiler] Address a couple of FIXMEs in the counters sampling code.

[profiler] Use eglib alloc/free functions everywhere.

[profiler] Warn if we can't detect the CPU count.

[profiler] Use the native thread wrapper functions to manage threads.

[utils/threads] Add and export a mono_native_thread_join () function.

[utils/threads] Export all native thread functions.

[profiler] Access in_shutdown atomically.

[profiler] Drop support for perf events on Linux.

[utils/lfa] Add a missing #include to fix the profiler build.

[profiler] Disable GC move events when allocation events are disabled.

They serve no purpose when allocation profiling is enabled, and just make log
files bigger than they have to be.

[profiler] Clean up and improve the helper thread code.

* Don't invoke undefined behavior if an fd is >= FD_SETSIZE.
* Allow multiple command sockets.
* Close the command socket(s) on shutdown.
* Properly run cleanup logic on shutdown.

[profiler] Process requests in method_jitted ().

[profiler] Remove the need to pass a MonoProfiler argument to some functions.

This is now available on the MonoProfilerThread structure.

[profiler] Fix the way the profiler frees hazardous data on shutdown.

[profiler] Replace a check with an assert in coverage_filter ().

[profiler] Add messages to most assertions.

[profiler] Remove the ign_res () hack.

[profiler] Clean up the #include mess in mono-profiler-log.c.

Half of these aren't even needed - get rid of those. Sort the rest.

[profiler] Remove mono_threads_attach_tools_thread () declaration.

We get this from mono-threads.h now.

[profiler] Remove some broken, commented-out code from register_method_local ().

[profiler] assert () -> g_assert ().

[profiler] Always enable the helper thread.

We need it in all cases now that we use it to periodically flush all buffers.
The only platforms it was disabled on was NaCl, which we haven't bothered to
support in a while anyway.

[profiler] Encapsulate some common logic in the ENTER_LOG/EXIT_LOG macros.

This also has the nice effect of reducing the amount of times we attempt to
take the reader lock. Now we take it and release it once for a single event.

[profiler] Add a missing deinit_thread () call to the helper thread.

[profiler] Fix several race conditions in accessing MonoProfilerThread.buffer.

Only in very few situations is it okay to access this field without first
taking the reader lock or exclusive lock.

Fixes some crashes observed in the profiler stress tests.

[profiler] Don't flush empty log buffers.

[profiler] Add an extra assert.

[sgen] Always report object moves on the GC thread.

This avoids exposing the SGen worker threads through the profiler APIs. Doing
so had two main problems:

* It's easy for a user of the profiler API to accidentally mess with the state
  of the worker thread in all sorts of ways, which could break the GC.
* It's been a long-standing guarantee that any profiler API callback will be
  invoked on a thread that is, at the very least, attached to the runtime via
  the mono-threads infrastructure. This is not the case for SGen workers.

This fixes a deadlock in the log profiler when using major=marksweep-conc.

[profiler] Rename decode.c to mprof-report.c.

[profiler] Rename proflog.{c,h} to mono-profiler-log.{c,h} for consistency with other profilers.

[profiler] Remove utils.c/utils.h, move the code to proflog.c/decode.c, and clean it up.

* Drop support for fast/null timers.
* Use Mono runtime functions for getting the thread ID and allocating buffers.

[acceptance-tests] Improve the profiler-stress test runner.

* Emit an NUnit test report file.
* Set a timeout of 6 hours per test.
* Do a thread dump for timed-out tests.
* Don't print stdout/stderr for passing tests.

[ci] Bump the timeout of the profiler stress test suite to 24 hours.

[sgen] Remove bogus mono_profiler_events assert.

This variable can be changed at any time by any thread. It makes no sense to
assert that it has a particular flag.

[utils] Move MONO_DEPRECATED to mono-publib.h and make it imply MONO_API + MONO_RT_EXTERNAL_ONLY.

Also make it work with MSVC.

[profiler] Remove the long-obsolete hsmode option.

Also refactor the heap walk code a bit.

[profiler] Document and fix the onlycoverage option.

[profiler] Remove the countersonly option.

It was never documented and is no longer functional.

[man] Update mprof-report.1 to indicate the correct default maxframes value.

[profiler] Use the signal shutdown logic for the sampling thread for more cases.

[profiler] Remove some indirection in setting up GC events.

* mono_gc_enable_events: This did nothing for SGen and we called it
  unconditionally for Boehm. So no point in having it around.
* mono_gc_enable_alloc_events: The performance difference between checking
  a static alloc_events variable versus checking mono_profiler_events is
  marginal at best and doesn't justify the indirection, especially now that
  we support managed allocators while profiling.

[profiler] Lift profiler limitation in mono_class_get_allocation_ftn ().

This has not actually been necessary since allocation reporting was moved into
the GC implementations.

[profiler] Don't leak counter data structures on shutdown.

[profiler] Simplify counters initialization.

[profiler] Don't emit a perf counters sample event if none are registered.

[profiler] Fix the code that synthesizes thread end events.

It could previously fail to process a newly allocated buffer.

[profiler] Add counters for all event types.

[metadata] Fix FullAOT build

The function was renamed in https://github.com/mono/mono/commit/a0ad8154f7cdca706802aa196d95686e0f8d3b7b#diff-9a532ce56888ac899468740a872fd7e0L1814, but this stub was missed.

[btls] Fix bashism in create-object-library.sh

Found by checkbashisms(1) as "unsafe echo with backslash". Some shells don't expand embedded \n in a string by default,
which broke the build on systems like ArchLinux or Alpine where /bin/sh points to those shells because the generated .lo
file contained just a single line instead of two.

Thanks to @JustArchi for researching this on the Mono Gitter channel, it made fixing the issue a lot easier!

[System]: ChainValidationHelper: ignore port number when validating a certificate's host name.

(cherry picked from commit 4d28eb4b889d4f317675998f906cd9ca47440566)
(cherry picked from commit 7c88bf755ec19ecd7e6db578de329567ee41b8d0)

[BTLS]: Ignore port number when validating certificates.

(cherry picked from commit baebc1d8c0e5ebe0e98f737390546b834baf2932)

[BTLS]: Send server name extension on the client.

(cherry picked from commit 2602c52de4773fe028b36d13d14555036835aa4e)

Fix build break from 81fb20737f810def6dde88d87a1b2f23cdb736c3

The HttpWebResponse stub that is used e.g. on the monotouch_watch profile was missing an overload that this commit was using.

[corlib] Fix a warning

Fix build break from efd53c8e2f1823568309a2920091b724be0feff8

The new/renamed TypeBuilderInstantiation wasn't available with FULL_AOT_RUNTIME,
breaking the monotouch profile build.

[System] Fill more WebException details when CONNECT fails. Fixes #19594

Bump corlib version for efd53c8e2f1823568309a2920091b724be0feff8.

[scripts]: Add 'btls-cert-sync' script.

(cherry picked from commit 30fabaedd3c8e7f6cc7b512ce731396bb82e98eb)

[System]: Fix MonoTlsProviderFactory.TryDynamicLoad() logic.

If the MONO_TLS_PROVIDER environment variable is missing, use "default".

(cherry picked from commit 7002264dd26784f8df2a5a4f39db768da892b9d1)

[SRE] Rename MonoGenericClass to TypeBuilderInstantiation to match MS. (#3679)

[runtime] Fix a warning.

[System]: Make BTLS the default TLS Provider on Desktop.

[cert-sync]: Make the new store the default and add '--legacy' for the old one.

[BTLS]: Remove some debugging methods.

Merge pull request #3681 from rolfbjarne/fix-monotouch-build

[System] Fix monotouch* build.

[System] Fix monotouch* build.

[System]: Enable AndroidPlatform.CertStoreLookup().

Uses reflection to call into AndroidEnvironment.CertStoreLookup()
(which as been added in https://github.com/xamarin/xamarin-android/pull/246).

Bump boringssl to fix the build

[btls] Only use cmake policies CMP0026/CMP0042 if cmake supports it

Some older cmake versions don't.

[System]: AndroidPlatform.CertStoreLookup(): add to the build, but return null.

This needs to have support added to xamarin-android before we can enable it.

(cherry picked from commit 9e46b5744b93bcc0bc36ab9d957a5510aaeabf1f)

Merge pull request #3676 from chamons/SignalHandlerAPI_XM45

Add Mono.Runtime to XM 4.5 profile

[BTLS]: Add managed BTLS sources.

* external/boringssl: unmanaged code from Google (added earlier).

* mono/btls: unmanaged BTLS lives here (added earlier).

* System/Mono.Btls: managed BTLS lives here.

* Mono.Btls.Interface: The purpose of this assembly is to allow test suites
  such as xamarin/web-tests to test parts of the BTLS APIs without making
  System.dll's internals visible.  It is not a stable and maintained API.

[Mono.Security]: Add the new certificate store.

BTLS uses a different format and naming scheme for the local certificate store.

Before using BTLS for the first time on desktop, you need to convert the local
certificate store by either using the new `btls-cert-sync` tool (which will convert
your existing certificate store to the new new format) or `cert-sync` with the
`--btls` argument.

The new format and naming scheme is identically to what you'll find in
'/system/etc/security/cacerts/' on an Android device.  It stores the certificates
in PEM format (with a human-readable text section added) using filenames which
are based on the certificate's subject name hash.

Previously, Mono stored the certificates in binary DER format, using a filename
based on either the Subject-Key-Identifier (if available) or the thumbprint.

We cannot access this from BTLS because computing these filenames requires
reading the entire X509 certificate (including the X509v3 extensions), but
their native lookup code only gives us the SubjectName.

This transitioning won't affect Android users as we're using the system
certificate store (including the user certificate store, via a Java callback).

* Mono.Security.X509.X509Store and X509Stores:
  Support both the old and the new format.  Using the new format requires BTLS.

* Mono.Security.X509.X509StoreManager:
  Add `NewCurrentUserPath` and `NewLocalMachinePath` properties for the
  new path names (~/.config/.mono/new-certs).

* Mono.Security.X509.X509StoreManager:
  Add `NewCurrentUser` and `NewLocalMachine` properties.  Requires BTLS.

* tools/security/cert-sync:
  Add new `--btls` argument to use the new store.

[System]: Add AndroidPlatform.CertStoreLookup().

This is conditional to HAVE_BTLS and needs to have support added
to xamarin-android first.

[System]: New X509Helper2 APIs for BTLS.

The managed HAVE_BTLS conditionals will be removed once BTLS itself has landed.

[System.Core] Cleanup more warnings

[System.Data] Cleanup warnings

[System.Data] Implements SqlClient async readers. Fixes #44843

[BTLS]: Add the native BTLS sources.

Merge pull request #3663 from lateralusX/jlorenss/win-api-family-support-libgcmonosgen

Build libgcmonosgen under none desktop Windows API family.

Build libgcmonosgen under none desktop Windows API family.

Initial work to build libgcmonosgen under none desktop Windows API families.
Changes are mainly related to header files included by libgcmonosgen sources and shared
by other libraries.

Sre cleanup9 (#3661)

* [SRE] Assign pseudo tokens to SRE objects in the BCL instead of doing it in the runtime.

Real tokens are assigned after the dynamic types they depend on have been created, by creating the corresponding finished reflection object
(i.e. a MonoMethod for a MethodOnTypeBuilderInst etc.) and obtaining its token.
In Save mode, this is done in ModuleBuilder.Save ().
In Run mode, this is done on-demand when the JIT tries to resolve a token into its corresponding metadata object, by the runtime calling
the RuntimeResolve () method on the builder object.

* [SRE] Remove dead code.

* [SRE] Remove the MonoGenericMethod/MonoGenericCMethod classes, they have not needed.

* [SRE] Delay the creation of MonoClass-es for MonoGenericClass/GenericTypeParameterBuilder/SymbolType until its needed.

* [SRE] Small code cleanup.

* [SRE] Remove the TypeBuilder.IsGenericParameter () icall, it should return false.

* [SRE] Unify the calling of the various RuntimeResolve () methods by adding a RuntimeResolve () method to ModuleBuilder, and calling that from native code.

* [SRE] Remove unused TypeBuilder::get_event_info () icall.

* [SRE] Remove more unused/empty icalls.

[Mono.Security]: Cleanup the MonoTlsProvider's certificate validation code. (#3674)

* [Mono.Security]: Remove Mono.Security.Interface.IMonoTlsContext.

This interface is not actually used anywhere anymore, so we can also
just get rid of it.

* [System]: Removed unused private TLS classes.

* [Mono.Security]: Cleanup the MonoTlsProvider's certificate validation code.

Each TLS Provider now has its own certificate validator and there is no
automatic fallback anymore.  We only have three different TLS Providers,
"legacy", "appletls" and "btls" - and legacy is the only one which used
the managed validator.  Both AppleTls and BTLS provider their own validator,
which invokes some native API.

All of the following were internal APIs.

* MonoTlsProvider.HasCustomSystemCertificateValidator has been removed.

* MonoTlsProvider.InvokeSystemCertificateValidator() has been renamed
  into ValidateCertificate() and is now abstract.

(cherry picked from commit 63c0803b2b60ccf50cc16ae93cc70443ccf364f2)

[mcs] Branch visited labels cache for conditional block. Fixes #44751

Add Mono.Runtime to XM 4.5 profile

- Extends https://github.com/mono/mono/pull/3624