cc: Do not add already failed image decode tasks as dependencies.

cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=606117
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/68f3b80bd5f76377e0b4bb067fd8e759d33cefc3
Cr-Commit-Position: refs/heads/master@{#389365}

[prashant.n, 2016-04-21 13:55:17]

Description was changed from

==========
cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=None
==========

to

==========
cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=None
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[prashant.n, 2016-04-21 13:56:38]

prashant.n@samsung.com changed reviewers:
+ ericrk@chromium.org, reveman@chromium.org, vmpstr@chromium.org

[prashant.n, 2016-04-21 13:56:39]

ptal.

[prashant.n, 2016-04-21 21:59:47]

ping

[ericrk, 2016-04-22 22:14:48]

LGTM - can you add a unit test to ensure that this behavior doesn't regress?

[prashant.n, 2016-04-23 02:24:14]

The CQ bit was checked by prashant.n@samsung.com

[commit-bot: I haz the power, 2016-04-23 02:24:27]

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

[commit-bot: I haz the power, 2016-04-23 02:24:28]

This CL has an open dependency (Issue 1903733003 Patch 60001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[prashant.n, 2016-04-23 04:14:24]

The CQ bit was checked by prashant.n@samsung.com

[prashant.n, 2016-04-23 04:14:25]

The patchset sent to the CQ was uploaded after l-g-t-m from ericrk@chromium.org
Link to the patchset: https://codereview.chromium.org/1909953002/#ps20001
(title: "changed patch dependency")

[commit-bot: I haz the power, 2016-04-23 04:14:34]

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

[prashant.n, 2016-04-23 04:18:33]

Description was changed from

==========
cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=None
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=606117
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-04-23 05:17:59]

Description was changed from

==========
cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=606117
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=606117
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-04-23 05:18:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-23 05:19:21]

Description was changed from

==========
cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=606117
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG=606117
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/68f3b80bd5f76377e0b4bb067fd8e759d33cefc3
Cr-Commit-Position: refs/heads/master@{#389365}
==========

[commit-bot: I haz the power, 2016-04-23 05:19:22]

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

[prashant.n, 2016-04-23 09:29:30]

On 2016/04/22 22:14:48, ericrk wrote:
> LGTM - can you add a unit test to ensure that this behavior doesn't regress?

In another CL, I'll add unit test.