Remove ToTimeoutSeconds.SendPingAsync and fix TimeSpan behavior

[deeprobin]

As described in #64860 (comment), this reverts the SendPingAsync overloads and further code review suggestions applied.

Follow-up for #64860 - Closes #14336

/cc @danmoseley
/cc @stephentoub

[ghost]

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

[ghost]

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

---

As described in #64860 (comment), this reverts the SendPingAsync overloads and further code review suggestions applied.

Follow-up for #64860 - Closes #14336

/cc @danmoseley
/cc @stephentoub

Author:	deeprobin
Assignees:	-
Labels:	area-System.Net, new-api-needs-documentation
Milestone:	-

[stephentoub]

Thanks.

[danmoseley]

Thanks could you please open a new issue specifically to implement the async Ping API? That will involve someone going into the ping implementation so it should be separate from #67156

[deeprobin]

Thanks could you please open a new issue specifically to implement the async Ping API? That will involve someone going into the ping implementation so it should be separate from #67156

#67260