`SocketAsyncEventArgs` cleanups and preparations for the real thing. (#16001)

Cleaning up the `SocketAsyncEventArgs` class and preparing it for it's eventual
adaption from CoreFX.

There are a few instance fields and property accessors with `internal` visibility,
which the CoreFX version does not have.  Replaced these with accessor methods.

The `PolicyRestricted` property and it's related `SocketAsyncEventArgs (bool)`
constructor is not used anywhere and the CoreFX version has a constructor with the
same signature, but different semantics.

In the CoreFX version, `Complete ()` does not call `DoComplete (this)` - renamed it
into `Complete_internal ()` to avoid any ambiguities and help the transitioning.

1 files changed, 1 insertions, 1 deletions

diff --git a/mcs/class/referencesource/System/net/System/Net/Sockets/Socket.cs b/mcs/class/referencesource/System/net/System/Net/Sockets/Socket.cs
index 78673562a9c..8ef3b670b51 100644
--- a/mcs/class/referencesource/System/net/System/Net/Sockets/Socket.cs
+++ b/mcs/class/referencesource/System/net/System/Net/Sockets/Socket.cs
@@ -7851,7 +7851,7 @@ namespace System.Net.Sockets {
             if (s_LoggingEnabled) Logging.Enter(Logging.Sockets, null, "ConnectAsync", "");
 
             // Throw if multiple buffers specified.
-            if (e.m_BufferList != null) {
+            if (e.BufferList != null) {
                 throw new ArgumentException(SR.GetString(SR.net_multibuffernotsupported), "BufferList");
             }