Use TestSimpleTaskRunner::NumPendingTasks instead of GetPendingTasks().size()

Use TestSimpleTaskRunner::NumPendingTasks instead of GetPendingTasks().size()

TestSimpleTaskRunner::GetPendingTasks returns a full copy of the queued
pending tasks, which is less clean, and also that prevents a refactoring
that makes tasks move-only.

This CL replaces them with an equivalent NumPendingTasks to avoid the copy.

BUG=554299
Committed: https://crrev.com/bdac06c0a9913c5bc245d9e4b900b6af826e503f
Cr-Commit-Position: refs/heads/master@{#418496}

[tzik, 2016-09-13 07:59:17]

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

[commit-bot: I haz the power, 2016-09-13 07:59:39]

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

[tzik, 2016-09-13 08:17:39]

Description was changed from

==========
Use TestSimpleTaskRunner::NumPendingTasks instead of GetPendingTasks().size()

BUG=
==========

to

==========
Use TestSimpleTaskRunner::NumPendingTasks instead of GetPendingTasks().size()

TestSimpleTaskRunner::GetPendingTasks returns a full copy of the queued
pending tasks, which is less clean, and also that prevents a refactoring
that makes tasks move-only.

This CL replaces them with an equivalent NumPendingTasks to avoid the copy.

BUG=554299
==========

[tzik, 2016-09-13 08:17:39]

tzik@chromium.org changed reviewers:
+ dcheng@chromium.org, rockot@chromium.org, stevenjb@chromium.org

[commit-bot: I haz the power, 2016-09-13 08:38:57]

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

[commit-bot: I haz the power, 2016-09-13 08:38:57]

Dry run: This issue passed the CQ dry run.

[tzik, 2016-09-13 08:41:20]

PTAL

[tzik, 2016-09-13 09:26:14]

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

[commit-bot: I haz the power, 2016-09-13 09:26:27]

Dry run: 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-13 09:48:53]

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

[commit-bot: I haz the power, 2016-09-13 09:48:54]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tzik, 2016-09-13 10:34:17]

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

[commit-bot: I haz the power, 2016-09-13 10:34:34]

Dry run: 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-13 11:47:42]

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

[commit-bot: I haz the power, 2016-09-13 11:47:43]

Dry run: This issue passed the CQ dry run.

[stevenjb, 2016-09-13 16:03:20]

lgtm

[Ken Rockot(use gerrit already), 2016-09-13 16:04:49]

lgtm

[dcheng, 2016-09-13 16:07:47]

lgtm

[tzik, 2016-09-14 05:08:02]

Patchset #5 (id:80001) has been deleted

[tzik, 2016-09-14 05:08:08]

The CQ bit was checked by tzik@chromium.org

[tzik, 2016-09-14 05:08:09]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org,
stevenjb@chromium.org, dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2335153002/#ps100001
(title: "rebase")

[commit-bot: I haz the power, 2016-09-14 05:08:20]

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-14 05:55:01]

Description was changed from

==========
Use TestSimpleTaskRunner::NumPendingTasks instead of GetPendingTasks().size()

TestSimpleTaskRunner::GetPendingTasks returns a full copy of the queued
pending tasks, which is less clean, and also that prevents a refactoring
that makes tasks move-only.

This CL replaces them with an equivalent NumPendingTasks to avoid the copy.

BUG=554299
==========

to

==========
Use TestSimpleTaskRunner::NumPendingTasks instead of GetPendingTasks().size()

TestSimpleTaskRunner::GetPendingTasks returns a full copy of the queued
pending tasks, which is less clean, and also that prevents a refactoring
that makes tasks move-only.

This CL replaces them with an equivalent NumPendingTasks to avoid the copy.

BUG=554299
==========

[commit-bot: I haz the power, 2016-09-14 05:55:03]

Committed patchset #5 (id:100001)

[commit-bot: I haz the power, 2016-09-14 05:57:00]

Description was changed from

==========
Use TestSimpleTaskRunner::NumPendingTasks instead of GetPendingTasks().size()

TestSimpleTaskRunner::GetPendingTasks returns a full copy of the queued
pending tasks, which is less clean, and also that prevents a refactoring
that makes tasks move-only.

This CL replaces them with an equivalent NumPendingTasks to avoid the copy.

BUG=554299
==========

to

==========
Use TestSimpleTaskRunner::NumPendingTasks instead of GetPendingTasks().size()

TestSimpleTaskRunner::GetPendingTasks returns a full copy of the queued
pending tasks, which is less clean, and also that prevents a refactoring
that makes tasks move-only.

This CL replaces them with an equivalent NumPendingTasks to avoid the copy.

BUG=554299

Committed: https://crrev.com/bdac06c0a9913c5bc245d9e4b900b6af826e503f
Cr-Commit-Position: refs/heads/master@{#418496}
==========

[commit-bot: I haz the power, 2016-09-14 05:57:01]

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