commit f554b3c577f69fa9ffad5c07155898c2d985ac76
author Tomas Gunnarsson <tommi@webrtc.org> Mon Feb 08 15:00:10 2021
committer Commit Bot <commit-bot@chromium.org> Mon Feb 08 17:52:01 2021
tree 2ae61f725dc74821553ef36a205b96fc060bab1d
parent f4fa763aee83e05457b7f911fe18805f0f29898c

Remove thread hops from events provided by JsepTransportController.

Events associated with Subscribe* methods in JTC had trampolines that
would use an async invoker to fire the events on the signaling thread.
This was being done for the purposes of PeerConnection but the concept
of a signaling thread is otherwise not applicable to JTC and use of
JTC from PC is inconsistent across threads (as has been flagged in
webrtc:9987).

This change makes all CallbackList members only accessible from the
network thread and moves the signaling thread related work over to
PeerConnection, which makes hops there more visible as well as making
that class easier to refactor for thread efficiency.

This CL removes the AsyncInvoker from JTC (webrtc:12339)

The signaling_thread_ variable is also removed from JTC and more thread
checks added to catch errors.

Bug: webrtc:12427, webrtc:11988, webrtc:12339
Change-Id: Id232aedd00dfd5403b2ba0ca147d3eca7c12c7c5
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/206062
Commit-Queue: Tommi <tommi@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33195}