tree 040c9a7e141324040565b64ae11c8c5eff9e8643
parent 204094d0977ff723faf1ce042017e45e3107b6a8
author Tim Detwiler <tjdetwiler@google.com> 1585951798 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1585951798 +0000

[audio_core] Simplify TLFunctions in OutputPipeline

We were doing some complex linking of TimelineFunctions between mix
stages. This is a hold-over from when all the mix stages ran in
lock-step (so we used the same TimelineFunction for all stages).

In reality, we can create new TimelineFunctions for the intermediate
stages instead. This makes things simpler and easier to reason about.
In addition this fixes a bug where loopback can be silent when running
mix stages at different rates.

Test: fx test -o audio_core_unittests
      > Added a new unit tests for Loopback + Upsample that fails before
        this patch is applied. We also improve these tests so that we
        don't verify the loopback streams against silence, which made
        it impossible to test that we were looping back properly.
Change-Id: I44df76662551ea1482efb6855c4793baea823618
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/377234
Commit-Queue: Tim Detwiler <tjdetwiler@google.com>
Reviewed-by: Martin Puryear <mpuryear@google.com>
Testability-Review: Martin Puryear <mpuryear@google.com>