Make Android/iOS local/remote description accessors thread safe.
Since the descriptions can be modified on the signaling thread,
ToString can only be safely called on that thread.
Bug: webrtc:11791
Change-Id: Icf6aada8aa66d00be94c6bda7b22e41b5d3bbc17
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/180541
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Reviewed-by: Anders Carlsson <andersc@webrtc.org>
Commit-Queue: Taylor <deadbeef@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31862}
diff --git a/api/peer_connection_interface.h b/api/peer_connection_interface.h
index b1beb84..09317b8 100644
--- a/api/peer_connection_interface.h
+++ b/api/peer_connection_interface.h
@@ -909,6 +909,10 @@
const std::string& label,
const DataChannelInit* config) = 0;
+ // NOTE: For the following 6 methods, it's only safe to dereference the
+ // SessionDescriptionInterface on signaling_thread() (for example, calling
+ // ToString).
+
// Returns the more recently applied description; "pending" if it exists, and
// otherwise "current". See below.
virtual const SessionDescriptionInterface* local_description() const = 0;
@@ -1136,6 +1140,14 @@
// thus the observer object can be safely destroyed.
virtual void Close() = 0;
+ // The thread on which all PeerConnectionObserver callbacks will be invoked,
+ // as well as callbacks for other classes such as DataChannelObserver.
+ //
+ // Also the only thread on which it's safe to use SessionDescriptionInterface
+ // pointers.
+ // TODO(deadbeef): Make pure virtual when all subclasses implement it.
+ virtual rtc::Thread* signaling_thread() const { return nullptr; }
+
protected:
// Dtor protected as objects shouldn't be deleted via this interface.
~PeerConnectionInterface() override = default;