cc: Implement test task runner for tests.

cc: Implement test task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

BUG=599863
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7c7060263d42c1b95874150f54635113e4b5f3e4
Cr-Commit-Position: refs/heads/master@{#396246}

[prashant.n, 2016-05-19 17:16:50]

Description was changed from

==========
cc: Implement simple task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

BUG=599863
==========

to

==========
cc: Implement simple task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

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

[prashant.n, 2016-05-19 17:18:14]

Description was changed from

==========
cc: Implement simple task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

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

to

==========
cc: Implement simple task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

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

[prashant.n, 2016-05-19 17:18:14]

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

[prashant.n, 2016-05-19 17:18:54]

PTAL. If looks okay, I'll extend to SoftwareImageDecodeControllerTest.

[ericrk, 2016-05-19 23:28:37]

Seems OK, but having this be a full class doesn't seem that useful. This could
work fine as a couple of static helper functions that just take the task
(although factoring them into their own file is nice).

[prashant.n, 2016-05-20 01:34:06]

On 2016/05/19 23:28:37, ericrk wrote:
> Seems OK, but having this be a full class doesn't seem that useful. This could
> work fine as a couple of static helper functions that just take the task
> (although factoring them into their own file is nice).

I'll modify as it adds lot of code.

[prashant.n, 2016-05-20 14:36:58]

ptal.

[ericrk, 2016-05-20 17:47:40]

lgtm

Thanks!

[vmpstr, 2016-05-20 23:23:19]

lgtm with a naming suggestion

https://codereview.chromium.org/1997743002/diff/40001/cc/test/simple_task_run...
File cc/test/simple_task_runner.h (right):

https://codereview.chromium.org/1997743002/diff/40001/cc/test/simple_task_run...
cc/test/simple_task_runner.h:14: class SimpleTaskRunner {
nit: can you rename this to something to indicate that it's a test? Any one of
the followings would work:

TestTileTaskRunner
FakeTileTaskRunner
TestTileTaskUtils

[prashant.n, 2016-05-24 04:04:16]

Description was changed from

==========
cc: Implement simple task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

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

to

==========
cc: Implement test task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

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

[reveman, 2016-05-25 03:58:59]

lgtm

[prashant.n, 2016-05-26 17:38:10]

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

[prashant.n, 2016-05-26 17:38:10]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org,
ericrk@chromium.org
Link to the patchset: https://codereview.chromium.org/1997743002/#ps120001
(title: "nits")

[commit-bot: I haz the power, 2016-05-26 17:38:39]

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

[commit-bot: I haz the power, 2016-05-26 19:00:24]

Description was changed from

==========
cc: Implement test task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

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

to

==========
cc: Implement test task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

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

[commit-bot: I haz the power, 2016-05-26 19:00:26]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-05-26 19:01:41]

Description was changed from

==========
cc: Implement test task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

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

to

==========
cc: Implement test task runner for tests.

This patch implements simple task runner which handles single task.
The functions provided in the class can be used for operating over
task.

BUG=599863
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7c7060263d42c1b95874150f54635113e4b5f3e4
Cr-Commit-Position: refs/heads/master@{#396246}
==========

[commit-bot: I haz the power, 2016-05-26 19:01:43]

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