• Glenn Kasten's avatar
    Bug 5244675 handle the switch cases of AAC decode · f5ff1a75
    Glenn Kasten authored
    Fix handling of AUDIOPLAYER_FROM_ADTS_ABQ_TO_PCM_BUFFERQUEUE decode use case:
      - Play::GetPosition: explicitly list decode case and return media
        player time; previously returned zero.
      - Seek::SetLoop for AudioPlayer or MediaPlayer: return
        SL_RESULT_FEATURE_UNSUPPORTED on non-URI data source.  This is
        correctly passed up to app.
    
    The remaining have no immediate impact, but should make future maintenance easier:
      - Play::GetDuration: explicitly list decode case and return unknown time;
        no impact as already covered by default.
      - Seek::SetPosition: explicitly list decode case and handle as no-op; no
        impact as already covered by default.
      - Seek::SetPosition for AudioPlayer or MediaPlayer: return
        SL_RESULT_FEATURE_UNSUPPORTED with unsupported data source e.g. buffer
        queue.  No impact as the result is currently ignored, but added a
        FIXME to pass this up to app.
      - AndroidBufferQueue::Clear: explicitly list decode case as a...
    f5ff1a75
handler_bodies.c 4.96 KB