V4L2VDA: do not queue output buffer during resolution change.

V4L2VDA: do not queue output buffer during resolution change.

This issue happens when a resolution change happens right
after a reset. First the client requests a reset. The image
processor is still processing some buffers and they will be
ignored when returning to V4L2VideoAccelerator. Then a
resolution change occurs and V4L2VDA stops the output queue.
The image processor returns a buffer and V4L2VDA queues it
back. Since the queue is stopped, STREAMON will be called to
start the queue. Finally REQBUFS will fail during resolution
change because the queue is not stopped.

The fix is we should not queue the buffer returned from
image proessor if a resolution change is in progress.

BUG=chrome-os-partner:55347
TEST=Seek when playing switch_1080p_720p.mp4 on oak.
The video is from gs://chromiumos-test-assets-public/Shaka-Dash.

Committed: https://crrev.com/dbc04d68e9460ff36423ce36afba9508a91cd8bf
Cr-Commit-Position: refs/heads/master@{#405991}

[wuchengli, 2016-07-18 07:02:27]

wuchengli@chromium.org changed reviewers:
+ kcwu@chromium.org

[kcwu, 2016-07-18 10:19:36]

lgtm

[wuchengli, 2016-07-18 12:04:59]

The CQ bit was checked by wuchengli@chromium.org

[commit-bot: I haz the power, 2016-07-18 12:05:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-18 14:18:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-18 14:18:14]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[wuchengli, 2016-07-18 14:57:18]

The CQ bit was checked by wuchengli@chromium.org

[commit-bot: I haz the power, 2016-07-18 14:57:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-18 15:06:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-18 15:09:28]

Description was changed from

==========
V4L2VDA: do not queue output buffer during resolution change.

This issue happens when a resolution change happens right
after a reset. First the client requests a reset. The image
processor is still processing some buffers and they will be
ignored when returning to V4L2VideoAccelerator. Then a
resolution change occurs and V4L2VDA stops the output queue.
The image processor returns a buffer and V4L2VDA queues it
back. Since the queue is stopped, STREAMON will be called to
start the queue. Finally REQBUFS will fail during resolution
change because the queue is not stopped.

The fix is we should not queue the buffer returned from
image proessor if a resolution change is in progress.

BUG=chrome-os-partner:55347
TEST=Seek when playing switch_1080p_720p.mp4 on oak.
The video is from gs://chromiumos-test-assets-public/Shaka-Dash.
==========

to

==========
V4L2VDA: do not queue output buffer during resolution change.

This issue happens when a resolution change happens right
after a reset. First the client requests a reset. The image
processor is still processing some buffers and they will be
ignored when returning to V4L2VideoAccelerator. Then a
resolution change occurs and V4L2VDA stops the output queue.
The image processor returns a buffer and V4L2VDA queues it
back. Since the queue is stopped, STREAMON will be called to
start the queue. Finally REQBUFS will fail during resolution
change because the queue is not stopped.

The fix is we should not queue the buffer returned from
image proessor if a resolution change is in progress.

BUG=chrome-os-partner:55347
TEST=Seek when playing switch_1080p_720p.mp4 on oak.
The video is from gs://chromiumos-test-assets-public/Shaka-Dash.

Committed: https://crrev.com/dbc04d68e9460ff36423ce36afba9508a91cd8bf
Cr-Commit-Position: refs/heads/master@{#405991}
==========

[commit-bot: I haz the power, 2016-07-18 15:09:29]

Patchset 1 (id:??) landed as
https://crrev.com/dbc04d68e9460ff36423ce36afba9508a91cd8bf
Cr-Commit-Position: refs/heads/master@{#405991}