GpuVideoDecoder now propagates errors during Flush()

GpuVideoDecoder now propagates errors during Flush()

Previously it was possible for GVD to be processing a Flush() but not
have no outstanding bitstream buffers in the VDA when it received a
NotifyError(). For example, when decoding single frame videos. In that
case it went into the error state but never propagated the error. Now,
if an error is received while in the flushing state, the error is
propagated by calling the decode cb corresponding to the EOS buffer.

BUG=647081
TEST=manually verified that AVDA decode errors now propagate to WMPI

Committed: https://crrev.com/99d991f6f7c0f480c0a5d8b2165daebe01cd25d5
Cr-Commit-Position: refs/heads/master@{#419001}

[watk, 2016-09-15 00:19:04]

The CQ bit was checked by watk@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-15 00:19:55]

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

[watk, 2016-09-15 00:21:14]

Description was changed from

==========
GpuVideoDecoder now propogates errors during Flush()

Previously it was possible for GVD to be processing a Flush() but not
have no outstanding bitstream buffers in the VDA when it received a
NotifyError(). For example, when decoding single frame videos. In that
case it went into the error state but never propagated the error. Now,
if an error is received while in the flushing state, the error is
propagated by calling the decode cb corresponding to the EOS buffer.

BUG=647081
TEST=manually verified that AVDA decode errors now propagate to WMPI
==========

to

==========
GpuVideoDecoder now propagates errors during Flush()

Previously it was possible for GVD to be processing a Flush() but not
have no outstanding bitstream buffers in the VDA when it received a
NotifyError(). For example, when decoding single frame videos. In that
case it went into the error state but never propagated the error. Now,
if an error is received while in the flushing state, the error is
propagated by calling the decode cb corresponding to the EOS buffer.

BUG=647081
TEST=manually verified that AVDA decode errors now propagate to WMPI
==========

[watk, 2016-09-15 00:21:14]

watk@chromium.org changed reviewers:
+ sandersd@chromium.org

[watk, 2016-09-15 18:56:37]

The CQ bit was checked by watk@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-15 18:57:11]

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

[sandersd (OOO until July 31), 2016-09-15 19:01:53]

lgtm

[watk, 2016-09-15 19:09:36]

The CQ bit was unchecked by watk@chromium.org

[watk, 2016-09-15 19:09:36]

The CQ bit was checked by watk@chromium.org

[commit-bot: I haz the power, 2016-09-15 19:09:54]

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-15 22:04:55]

Description was changed from

==========
GpuVideoDecoder now propagates errors during Flush()

Previously it was possible for GVD to be processing a Flush() but not
have no outstanding bitstream buffers in the VDA when it received a
NotifyError(). For example, when decoding single frame videos. In that
case it went into the error state but never propagated the error. Now,
if an error is received while in the flushing state, the error is
propagated by calling the decode cb corresponding to the EOS buffer.

BUG=647081
TEST=manually verified that AVDA decode errors now propagate to WMPI
==========

to

==========
GpuVideoDecoder now propagates errors during Flush()

Previously it was possible for GVD to be processing a Flush() but not
have no outstanding bitstream buffers in the VDA when it received a
NotifyError(). For example, when decoding single frame videos. In that
case it went into the error state but never propagated the error. Now,
if an error is received while in the flushing state, the error is
propagated by calling the decode cb corresponding to the EOS buffer.

BUG=647081
TEST=manually verified that AVDA decode errors now propagate to WMPI
==========

[commit-bot: I haz the power, 2016-09-15 22:04:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-15 22:07:58]

Description was changed from

==========
GpuVideoDecoder now propagates errors during Flush()

Previously it was possible for GVD to be processing a Flush() but not
have no outstanding bitstream buffers in the VDA when it received a
NotifyError(). For example, when decoding single frame videos. In that
case it went into the error state but never propagated the error. Now,
if an error is received while in the flushing state, the error is
propagated by calling the decode cb corresponding to the EOS buffer.

BUG=647081
TEST=manually verified that AVDA decode errors now propagate to WMPI
==========

to

==========
GpuVideoDecoder now propagates errors during Flush()

Previously it was possible for GVD to be processing a Flush() but not
have no outstanding bitstream buffers in the VDA when it received a
NotifyError(). For example, when decoding single frame videos. In that
case it went into the error state but never propagated the error. Now,
if an error is received while in the flushing state, the error is
propagated by calling the decode cb corresponding to the EOS buffer.

BUG=647081
TEST=manually verified that AVDA decode errors now propagate to WMPI

Committed: https://crrev.com/99d991f6f7c0f480c0a5d8b2165daebe01cd25d5
Cr-Commit-Position: refs/heads/master@{#419001}
==========

[commit-bot: I haz the power, 2016-09-15 22:07:59]

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