[Reading List] Check web state after the distillation delay.

[Reading List] Check web state after the distillation delay.

It is possible that the web state has been released before the timeout
of the timer.
If this is the case, it means that the distillation was aborted.
Early return in that case.

There is no Cancelable delayed task. As the current situation is simple (only one
task is running at a time), store the ID of the task, and increment it when cancelling.
This is inspired by RunIfNotCanceled, but adapted to this much simpler situation.
Check the current ID when the task runs. If the ID was incremented, the task is cancelled.

BUG=695493
Review-Url: https://codereview.chromium.org/2717483003
Cr-Commit-Position: refs/heads/master@{#452540}
Committed: https://chromium.googlesource.com/chromium/src/+/d7e66510bedbf95d90337071155f8b07036cb002

[Olivier, 2017-02-23 16:11:44]

olivierrobin@chromium.org changed reviewers:
+ jif@chromium.org

[Olivier, 2017-02-23 17:17:32]

Description was changed from

==========
[Reading List] Check web state after the distillation delay.

It is possible that the web state has been released before the timeout
of the timer.
If this is the case, it means that the distillation was aborted.
Early return in that case.

BUG=695493
==========

to

==========
[Reading List] Check web state after the distillation delay.

It is possible that the web state has been released before the timeout
of the timer.
If this is the case, it means that the distillation was aborted.
Early return in that case.

There is no Cancelable delayed task. As the current situation is simple (only
one
task is running at a time), store the ID of the task, and increment it when
cancelling.
Check the current ID when the task runs. If the ID was incremented, the task is
cancelled.

BUG=695493
==========

[Olivier, 2017-02-23 17:18:35]

Description was changed from

==========
[Reading List] Check web state after the distillation delay.

It is possible that the web state has been released before the timeout
of the timer.
If this is the case, it means that the distillation was aborted.
Early return in that case.

There is no Cancelable delayed task. As the current situation is simple (only
one
task is running at a time), store the ID of the task, and increment it when
cancelling.
Check the current ID when the task runs. If the ID was incremented, the task is
cancelled.

BUG=695493
==========

to

==========
[Reading List] Check web state after the distillation delay.

It is possible that the web state has been released before the timeout
of the timer.
If this is the case, it means that the distillation was aborted.
Early return in that case.

There is no Cancelable delayed task. As the current situation is simple (only
one
task is running at a time), store the ID of the task, and increment it when
cancelling.
This is inspired by RunIfNotCanceled, but adapted to this much simpler
situation.
Check the current ID when the task runs. If the ID was incremented, the task is
cancelled.

BUG=695493
==========

[jif, 2017-02-23 17:32:10]

The CQ bit was checked by jif@chromium.org

[jif, 2017-02-23 17:32:10]

lgtm

[commit-bot: I haz the power, 2017-02-23 17:32:28]

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, 2017-02-23 17:43:36]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487871130147280,
"parent_rev": "ee09c859d3f509fc33d7c9e3b3ffe0959644f9e2", "commit_rev":
"d7e66510bedbf95d90337071155f8b07036cb002"}

[commit-bot: I haz the power, 2017-02-23 17:44:08]

Description was changed from

==========
[Reading List] Check web state after the distillation delay.

It is possible that the web state has been released before the timeout
of the timer.
If this is the case, it means that the distillation was aborted.
Early return in that case.

There is no Cancelable delayed task. As the current situation is simple (only
one
task is running at a time), store the ID of the task, and increment it when
cancelling.
This is inspired by RunIfNotCanceled, but adapted to this much simpler
situation.
Check the current ID when the task runs. If the ID was incremented, the task is
cancelled.

BUG=695493
==========

to

==========
[Reading List] Check web state after the distillation delay.

It is possible that the web state has been released before the timeout
of the timer.
If this is the case, it means that the distillation was aborted.
Early return in that case.

There is no Cancelable delayed task. As the current situation is simple (only
one
task is running at a time), store the ID of the task, and increment it when
cancelling.
This is inspired by RunIfNotCanceled, but adapted to this much simpler
situation.
Check the current ID when the task runs. If the ID was incremented, the task is
cancelled.

BUG=695493

Review-Url: https://codereview.chromium.org/2717483003
Cr-Commit-Position: refs/heads/master@{#452540}
Committed:
https://chromium.googlesource.com/chromium/src/+/d7e66510bedbf95d90337071155f...
==========

[commit-bot: I haz the power, 2017-02-23 17:44:10]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/d7e66510bedbf95d90337071155f...