commit 8408c9938cb543a7cd5e7127927f3b4d0834ad59
author Tomas Gunnarsson <tommi@webrtc.org> Sun Feb 14 13:19:12 2021
committer Commit Bot <commit-bot@chromium.org> Mon Feb 15 18:08:17 2021
tree ffb6ead6d2e7299b46706fcd5cdae35250a897a5
parent a33f41bf5812acbccd4f5368cfebb2f4352a5cee

Remove 'secondary sink' concept from webrtc::VideoReceiveStream.

In practice, support for multiple sinks is not needed and supporting
the API that allows for dynamically adding/removing sinks at runtime,
adds to the complexity of the implementation.

This CL removes that Add/Remove methods for secondary sinks as well
as vectors of callback pointers (which were either of size 0 or 1).
Instead, an optional callback pointer is added to the config struct
for VideoReceiveStream, that an implementation can consider to be
const and there's not a need to do thread synchronization for that
pointer for every network packet.

As part of webrtc:11993, this simplifies the work towards keeping
the processing of network packets on the network thread. The secondary
sinks, currently operate on the worker thread.

Bug: webrtc:11993
Change-Id: I10c473e57d3809527a1b689f4352e903a4c78168
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/207421
Commit-Queue: Tommi <tommi@webrtc.org>
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33272}