tree 60a72c304cc082e773aea6f7237ae32670cb3164
parent f1420cae27a7b3209d754e2e1363ee2ba290ffbd
author martinduke <martinduke@google.com> 1783528352 -0700
committer Copybara-Service <copybara-worker@google.com> 1783528400 -0700

Refactor callbacks and request lifetime.

Migrating SUBSCRIBE to a bidi stream has revealed several problems with object lifetimes and a complex web of callbacks. This revises the architecture in advance of the bidi migration.

* MoqtBidiStreamBase has a pure-virtual Detach() function that frees the session of any state related to the request, called when the stream is Reset and other FINed events (like REQUEST_ERROR). Waiting for Reset streams to be cleaned up in the QUIC stack was creating problems for reconnect test cases, and this is more thoughtful about when resources should be freed.

* BidiStreamDeletedCallback was not useful because it has no arguments. Now, each Bidi stream type has its own bespoke callbacks for deletion, and (for response streams) to register the key properties with the session in the first place.

* The specifics of what properties to use as keys are moved back into MoqtSession. The callbacks just present a pointer to the value/context. For example, MoqtNamespaceStream no longer has a notion of SubscribeNamespaceTree. It just asks the Session if a SUBSCRIBE_NAMESPACE is allowed.

* Add a static function for MoqtSession to check if it's still alive, and use it in callbacks generated by the session.

* Condense AI test bloat in moqt_publish_stream_test.

PiperOrigin-RevId: 944543487