UWP: System.Net.Sockets

[CIPop]

Tracking work for System.Net.Sockets in UWP

[caesar-chen]

I'm doing the final revision for S.N.Sockets Functional tests, and I have a question. For LoggingTests, I am not able to find an issue to disable it (I was using this epic number before). What number should I use?
dotnet/corefx#21019 (comment)

@davidsh @CIPop

[CIPop]

@Caesar1995 I think it's dotnet/corefx#20470

[caesar-chen]

Status update:

• Blocking issues: dotnet/corefx#20470 Resources File in .NET Standard (1.4) with New csproj #21158 Add SqlClient test cases for MARS over Named Pipes #21391

• Followed the checklist, we are not able to remove dependencies on ContextAwareResult.Uap.cs and CompletionPortHelper.Uap.cs. They are important components for Sockets synchronous completion support.

I will start UWP: System.Net.NetworkInformation work.

UPDATE: dotnet/corefx#21158 has been fixed
UPDATE2: dotnet/corefx#21391 is a dupe of dotnet/corefx#20470
UPDATE3: Delete the not needed PAL files

[caesar-chen]

Sorry I made a mistake, CompletionPortHelper.Uap.cs cannot be removed, while ContextAwareResult.Uap.cs can be removed.

Analysis:
For SafeCloseSocket Windows' implementation, we need CompletionPortHelper.SkipCompletionPortOnSuccess() to decide whether we need to disable completions for synchronous success. [CompletionPortHelper.SkipCompletionPortOnSuccess() is using an api (Interop.Kernel32.SetFileCompletionNotificationModes) which has not been approved for UAP yet. That's why we need the PAL for CompletionPortHelper.

While ContextAwareResult.Uap.cs doesn't has this problem and can be removed.

I will submit a PR to correct my mistake.

cc: @CIPop @davidsh @DavidGoll

[CIPop]

SetFileCompletionNotificationModes which has not been approved for UAP yet

@Caesar1995 @DavidGoll I believe the synchronous completion mode for Sockets enhancement was added after we had the UAP profile discussions and analyzed the API surface.
Let's follow-up offline. Ideally we should have the same implementation for both Windows Desktop and UAP.

[caesar-chen]

I wasn't aware that the API has been approved.

Last time I built the project without the CompletionPortHelper.Uap.cs in UAP mode, the compiler complained that "error: SetFileCompletionNotificationModes() is not supported on UAP". But now the error goes away. I will delete both of the files.