Revert "Move signaling safety flag into SctpDataChannel and clarify its purpose"

This reverts commit 81d438a4fb11bc0de35229bd3894db6712935319.

Reason for revert: Breaks downstream tests.

Original change's description:
> Move signaling safety flag into SctpDataChannel and clarify its purpose
>
> Encapsulate the signaling thread safety flag within SctpDataChannel and
> rename to `controller_safety` to clarify its purpose. Remove details
> about the flag from the CreateProxy() interface. The observer previously
> received this flag via the CreateProxy method, but the observer is
> associated with the channel (not the proxy). Basically, the proxy
> creation layer should not need to know about the flag mechanism and the
> channel object was out of the loop, so this was a bit awkward.
>
> Bug: webrtc:510487699
> Change-Id: I57638ade52305b6479a9a9165747336c679631cd
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/304841
> Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
> Commit-Queue: Tomas Gunnarsson <tommi@webrtc.org>
> Cr-Commit-Position: refs/heads/main@{#47649}

Bug: webrtc:510487699
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I363214309a2e98359c4d9c63660b858413264ce9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/470560
Bot-Commit: rubber-stamper@appspot.gserviceaccount.com <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Johannes Kron <kron@webrtc.org>
Owners-Override: Johannes Kron <kron@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#47651}