V4L2SVDA: Move allocation from GPU Child thread to decoder thread.

V4L2SVDA: Move allocation from GPU Child thread to decoder thread.

Allocation may take time, and we don't need to be on the GPU Child
thread to do it. This also removes the explicit synchronization between
the decoder and GPU Child threads during picture set change.

BUG=574241
TEST=vdatest,crosvideo.appspot

Committed: https://crrev.com/6b8b26b67eeedd5fd40a3a2ca027a734cca4f218
Cr-Commit-Position: refs/heads/master@{#376120}

[Pawel Osciak, 2016-01-29 05:32:05]

Description was changed from

==========
V4L2SVDA: Move allocation from GPU Child thread to decoder thread.

Allocation may take time, and we don't need to be on the GPU Child
thread to do it. This also removes the explicit synchronization between
the decoder and GPU Child threads during picture set change.

BUG=574241
TEST=vdatest,crosvideo.appspot
==========

to

==========
V4L2SVDA: Move allocation from GPU Child thread to decoder thread.

Allocation may take time, and we don't need to be on the GPU Child
thread to do it. This also removes the explicit synchronization between
the decoder and GPU Child threads during picture set change.

BUG=574241
TEST=vdatest,crosvideo.appspot
==========

[Pawel Osciak, 2016-01-29 05:32:05]

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

[Pawel Osciak, 2016-02-15 01:24:48]

PTAL. PostProcessPendingEvents() not needed/used anymore.

[wuchengli, 2016-02-18 08:03:51]

lgtm

[Pawel Osciak, 2016-02-18 08:06:17]

The CQ bit was checked by posciak@chromium.org

[wuchengli, 2016-02-18 08:06:48]

https://codereview.chromium.org/1643123003/diff/20001/content/common/gpu/medi...
File content/common/gpu/media/v4l2_slice_video_decode_accelerator.cc (right):

https://codereview.chromium.org/1643123003/diff/20001/content/common/gpu/medi...
content/common/gpu/media/v4l2_slice_video_decode_accelerator.cc:979:
ScheduleDecodeBufferTaskIfNeeded();
I asked offline about why we need line 976-99 now and not before. The reason is
because we removed it from each Finish*().

https://codereview.chromium.org/1643123003/diff/20001/content/common/gpu/medi...
content/common/gpu/media/v4l2_slice_video_decode_accelerator.cc:1440:
&V4L2SliceVideoDecodeAccelerator::AssignPictureBuffersTask,
For the record, we had an offline discussion. I asked if we can call
CreateEGLImages here so we can save a roundtrip from child thread to decode
thread. The answer is no because CreateEGLImage needs the fds from
AssignPictureBuffersTask.

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

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1643123003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1643123003/20001

[commit-bot: I haz the power, 2016-02-18 08:45:57]

Description was changed from

==========
V4L2SVDA: Move allocation from GPU Child thread to decoder thread.

Allocation may take time, and we don't need to be on the GPU Child
thread to do it. This also removes the explicit synchronization between
the decoder and GPU Child threads during picture set change.

BUG=574241
TEST=vdatest,crosvideo.appspot
==========

to

==========
V4L2SVDA: Move allocation from GPU Child thread to decoder thread.

Allocation may take time, and we don't need to be on the GPU Child
thread to do it. This also removes the explicit synchronization between
the decoder and GPU Child threads during picture set change.

BUG=574241
TEST=vdatest,crosvideo.appspot
==========

[commit-bot: I haz the power, 2016-02-18 08:45:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-18 08:47:23]

Description was changed from

==========
V4L2SVDA: Move allocation from GPU Child thread to decoder thread.

Allocation may take time, and we don't need to be on the GPU Child
thread to do it. This also removes the explicit synchronization between
the decoder and GPU Child threads during picture set change.

BUG=574241
TEST=vdatest,crosvideo.appspot
==========

to

==========
V4L2SVDA: Move allocation from GPU Child thread to decoder thread.

Allocation may take time, and we don't need to be on the GPU Child
thread to do it. This also removes the explicit synchronization between
the decoder and GPU Child threads during picture set change.

BUG=574241
TEST=vdatest,crosvideo.appspot

Committed: https://crrev.com/6b8b26b67eeedd5fd40a3a2ca027a734cca4f218
Cr-Commit-Position: refs/heads/master@{#376120}
==========

[commit-bot: I haz the power, 2016-02-18 08:47:24]

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