Added support for Decode() calls during Reset().

Added support for Decode() calls during Reset().

Decode() calls are now queued for later processing when the decoder is in
RESETTING state.
Also output buffers are queued by OVDA during the Flush() triggered by Reset()
for re-use as soon as the Reset is done.

BUG=none
TEST=ovdatest, gles2, trybots

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=94700

[Ami GONE FROM CHROMIUM, 2011-07-28 18:43:57]

vrk: please review
piman: FYI since you were in on related email threads

[vrk (LEFT CHROMIUM), 2011-07-29 17:06:44]

LGTM

As we talked about in person, the OMX unit test is teetering on the edge of
needing a big refactor, but seems best to wait for now since API changes are
still being made/decode edge cases are still being discovered.

http://codereview.chromium.org/7524032/diff/1/content/common/gpu/media/omx_vi...
File content/common/gpu/media/omx_video_decode_accelerator.h (right):

http://codereview.chromium.org/7524032/diff/1/content/common/gpu/media/omx_vi...
content/common/gpu/media/omx_video_decode_accelerator.h:183: std::vector<int>
queued_picture_buffer_ids_;
nit: Add comment to explain queued picture buffers too (or elaborate on above
comment)

http://codereview.chromium.org/7524032/diff/1/content/common/gpu/media/omx_vi...
File content/common/gpu/media/omx_video_decode_accelerator_unittest.cc (right):

http://codereview.chromium.org/7524032/diff/1/content/common/gpu/media/omx_vi...
content/common/gpu/media/omx_video_decode_accelerator_unittest.cc:624:
encoded_data_next_pos_to_decode_ = 0;
Comment that we start playback over from the beginning on midstream reset?

[Ami GONE FROM CHROMIUM, 2011-07-29 17:17:09]

Thanks.

http://codereview.chromium.org/7524032/diff/1/content/common/gpu/media/omx_vi...
File content/common/gpu/media/omx_video_decode_accelerator.h (right):

http://codereview.chromium.org/7524032/diff/1/content/common/gpu/media/omx_vi...
content/common/gpu/media/omx_video_decode_accelerator.h:183: std::vector<int>
queued_picture_buffer_ids_;
On 2011/07/29 17:06:44, Victoria Kirst wrote:
> nit: Add comment to explain queued picture buffers too (or elaborate on above
> comment)

Done.

http://codereview.chromium.org/7524032/diff/1/content/common/gpu/media/omx_vi...
File content/common/gpu/media/omx_video_decode_accelerator_unittest.cc (right):

http://codereview.chromium.org/7524032/diff/1/content/common/gpu/media/omx_vi...
content/common/gpu/media/omx_video_decode_accelerator_unittest.cc:624:
encoded_data_next_pos_to_decode_ = 0;
On 2011/07/29 17:06:44, Victoria Kirst wrote:
> Comment that we start playback over from the beginning on midstream reset?

Done.