tree f7295c0135cdedfb8d03bd83a53085268f0d8bc2
parent 39e93b04ca74a453be18757d39065390c3251979
author Dale Sather <dalesat@google.com> 1588038164 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1588038164 +0000

[mediaplayer] fix issues preventing opus playback

This CL addresses the following issues, which otherwise prevent
opus playback from working:
1) The opus decoder produces interleaved output (in constrast to most
   ffmpeg audio decoders), but it allocates buffers 6 times as large
   as the generated packet (AVFrame, in ffmpeg terminology). This
   creates a situation in which the VMO used by the audio renderer is
   quickly exhausted. The ffmpeg audio decoder wrapper was modified
   to allocate system memory for the decoder and then copy only the
   relevant part of the payload buffer into a buffer allocated from
   the provided allocator (the one managing the renderer's VMO, in
   this case).
2) ffmpeg was complaining that the codec context's pkt_timebase field
   was not set. Though this had no audible effect, this CL fixes that
   anyway.
3) The AudioConsumer test UnsupportedCodec was using the opus codec
   designation, which is now supported. That designation was replaced
   with a nonsense string.

TEST: added an end-to-end test for opus
      $ fx test mediaplayer_tests
      The new test will fail until opus integration into the ffmpeg
      prebuilt is complete.

Bug: 50035

Change-Id: I9d854253222224490a48e497082ae24510bd4eb7
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/380304
Commit-Queue: Dale Sather <dalesat@google.com>
Reviewed-by: Austin Foxley <afoxley@google.com>
Testability-Review: Austin Foxley <afoxley@google.com>