commit d264fb81b9166323779bffa4b43c74726f6f6ef2
author Chih-Yu Huang <akahuang@google.com> Mon Sep 14 14:49:11 2020 +0900
committer Chih-Yu Huang <akahuang@google.com> Tue Sep 15 18:15:38 2020 +0900
tree f1d820d63c283de3ee052f05c3ac26e7fb01c8dc
parent 7b55edc6bacc315a2aa0a52cd4dcc6cf066a8356

V4L2Decoder: do not stop fetching output buffers on drain or flush

Originally we stop fetching output buffers when drain or flush is
finished, and resume fetching when getting resolution change event.
But the V4L2 driver might not send the event after drain or flush.

This CL changes not to stop fetching output buffers after drain or
flush.

Bug: 168557465
Test: Pass android.media.cts.AdaptivePlaybackTest#testVP8_eosFlushSeek
      with related CLs

Change-Id: I389b85c1718ae82933329a10fe09ca6abbb57f5d

components/V4L2Decoder.cpp

diff --git a/components/V4L2Decoder.cpp b/components/V4L2Decoder.cpp
index c070d64..0df4e50 100644
--- a/components/V4L2Decoder.cpp
+++ b/components/V4L2Decoder.cpp
@@ -502,8 +502,6 @@
     ALOG_ASSERT(mTaskRunner->RunsTasksInCurrentSequence());
     ALOG_ASSERT(mVideoFramePool, "mVideoFramePool is null, haven't get the instance yet?");
 
-    if (mState == State::Idle) return;
-
     if (mOutputQueue->FreeBuffersCount() == 0) {
         ALOGD("No free V4L2 output buffers, ignore.");
         return;