V4L2VideoDecodeAccelerator: fix frame returned after processor destroys.

V4L2VideoDecodeAccelerator: fix frame returned after processor destroys.

When V4L2VideoDecodeAccelerator sends a frame to processor, it uses an
extra BindToCurrentLoop. That is not required because the processor is
only accessed on the decoder thread. The worse thing is the processed
frame may arrive after the processor is destroyed because of the extra
hop, which may cause a native crash in V4L2VDA.

BUG=645854
BUG=b/29059119
TEST=Run VDA unittest and video_VideoSeek test. Play video.

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

[wuchengli, 2016-09-12 05:25:20]

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

[wuchengli, 2016-09-12 05:25:21]

Kuang-che. One line fix. PTAL.

[kcwu, 2016-09-12 08:30:04]

lgtm

BTW, |child_task_runner_| in V4L2ImageProcessor needs rename since it is not
"child task runner" for the case of V4L2VideoDecodeAccelerator.

[wuchengli, 2016-09-12 08:31:06]

On 2016/09/12 08:30:04, kcwu wrote:
> lgtm
> 
> BTW, |child_task_runner_| in V4L2ImageProcessor needs rename since it is not
> "child task runner" for the case of V4L2VideoDecodeAccelerator.
I'll upload a separate CL for that.

[wuchengli, 2016-09-12 08:31:10]

The CQ bit was checked by wuchengli@chromium.org

[commit-bot: I haz the power, 2016-09-12 08:31:22]

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-09-12 09:42:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-12 09:44:23]

Description was changed from

==========
V4L2VideoDecodeAccelerator: fix frame returned after processor destroys.

When V4L2VideoDecodeAccelerator sends a frame to processor, it uses an
extra BindToCurrentLoop. That is not required because the processor is
only accessed on the decoder thread. The worse thing is the processed
frame may arrive after the processor is destroyed because of the extra
hop, which may cause a native crash in V4L2VDA.

BUG=645854
BUG=b/29059119
TEST=Run VDA unittest and video_VideoSeek test. Play video.
==========

to

==========
V4L2VideoDecodeAccelerator: fix frame returned after processor destroys.

When V4L2VideoDecodeAccelerator sends a frame to processor, it uses an
extra BindToCurrentLoop. That is not required because the processor is
only accessed on the decoder thread. The worse thing is the processed
frame may arrive after the processor is destroyed because of the extra
hop, which may cause a native crash in V4L2VDA.

BUG=645854
BUG=b/29059119
TEST=Run VDA unittest and video_VideoSeek test. Play video.

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

[commit-bot: I haz the power, 2016-09-12 09:44:23]

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