Revises RequestCoordinator to not try to schedule another request when previous one fails.

Revises RequestCoordinator to not try to schedule another request when previous one fails.
This helps stop a failed request loop within single OfflinerTask run.
Also adds test to confirm failed requests are not removed from the queue (subject to some retry limit in future).

BUG=610521
Committed: https://crrev.com/272edd820eb1d150bfdc2081729877364c061447
Cr-Commit-Position: refs/heads/master@{#400544}

[dougarnett, 2016-06-17 22:21:59]

dougarnett@chromium.org changed reviewers:
+ fgorski@chromium.org, petewil@chromium.org

[dougarnett, 2016-06-17 22:22:01]

I think this is good change in general - but also solves a looping failed
request problems I experienced for single Task run.

[Pete Williamson, 2016-06-17 22:39:17]

lgtm

[dougarnett, 2016-06-17 23:07:33]

The CQ bit was checked by dougarnett@chromium.org

[commit-bot: I haz the power, 2016-06-17 23:08:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2071403003/1

[commit-bot: I haz the power, 2016-06-17 23:33:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-17 23:35:48]

Description was changed from

==========
Revises RequestCoordinator to not try to schedule another request when previous
one fails.
This helps stop a failed request loop within single OfflinerTask run.
Also adds test to confirm failed requests are not removed from the queue
(subject to some retry limit in future).

BUG=610521
==========

to

==========
Revises RequestCoordinator to not try to schedule another request when previous
one fails.
This helps stop a failed request loop within single OfflinerTask run.
Also adds test to confirm failed requests are not removed from the queue
(subject to some retry limit in future).

BUG=610521

Committed: https://crrev.com/272edd820eb1d150bfdc2081729877364c061447
Cr-Commit-Position: refs/heads/master@{#400544}
==========

[commit-bot: I haz the power, 2016-06-17 23:35:49]

Patchset 1 (id:??) landed as
https://crrev.com/272edd820eb1d150bfdc2081729877364c061447
Cr-Commit-Position: refs/heads/master@{#400544}