commit ee83f76be7c7c74cdda551596630a469710dd647
author Dale Sather <dalesat@google.com> Thu Apr 25 18:06:32 2019 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Thu Apr 25 18:06:32 2019 +0000
tree 69dffa7288c0fe53117777ba2eb89ad490f81cdf
parent 0b1cef5559c6e318367f350c49d0055bd15787b0

[mediaplayer] don't panic if the audio render stalls

This CL changes the behavior of the media player when the audio
renderer holds on to packets too long, which generally happens due to
scheduling issues. Because the player has allocated a limited amount
of space for transferring packets between itself and the renderer,
that space can be exhausted if the renderer doesn't retire packets
quickly enough.

The fix is to drop packets instead of panicking. There are a few
paths that had to be fixed to enable this. Sometimes the memory
allocation failures occur in a callback from the audio decoder.
Sometimes audio has to be interleaved after the decoder, and the
allocation failure is detected there. Sometimes (when no decoder
is needed) audio needs to be copied from system memory into vmo
memory, and the failure happens prior to the copy.

Two tests have been added to replicate these failure modes. Those
tests produced panics before the fixes were introduced.

TEST: new CQ tests
Change-Id: I26582d19481c6716f6a4141e6717b68670ea44f1