tree 7584d345b5065707374bc0607ab418eaf709da34
parent dc02e7f61fb614c44ebe2f8a44672f2ecb25e321
author martinduke <martinduke@google.com> 1770224991 -0800
committer Copybara-Service <copybara-worker@google.com> 1770225044 -0800

Make improvements to MoqtNamespaceStream.

Apologies for having another CL on this topic, but there are several fixes here:
- Setting the ObjectsAvailableCallback is no longer a separate operation in the NamespaceTask. This is an anti-pattern from FETCH that is suboptimal and causes problems when retrieving namespaces from the relay tree.

- IncomingSubscribeNamespaceCallback took std::optional<parameters> because it was also invoked on Unsubscribe. Now that is achieved by destroying the task, so Parameters are always present.

- There was a mistake in the test where task_ was assigned twice.

- There was no actual backpressure in the ObjectsAvailableCallback implementation.

- Got rid of std::move() when invoking multiuse callbacks.

PiperOrigin-RevId: 865435838