Worker: Replace CancellableTaskFactory with postDelayedCancellableTask

Worker: Replace CancellableTaskFactory with postDelayedCancellableTask

Before this CL, WorkerThread::ForceTerminationTask is implemented with
CancellableTaskFactory that is deprecated in favor of
WebTaskRunner::post(Delayed)CancellableTask.

This CL removes ForceTerminationTask/CancellableTaskFactory, and instead
implements the same functionality with postDelayedCancellableTask.

BUG=663624
Committed: https://crrev.com/fb4c1fe6ad37005d9818c65a42728ac7179bf499
Cr-Commit-Position: refs/heads/master@{#431559}

[nhiroki, 2016-11-10 07:23:56]

Description was changed from

==========
Worker: Replace CancellableTaskFactory with postDelayedCancellableTask

WorkerThread::ForceTerminationTask is currently implemented with
CancellableTaskFactory that is deprecated in favor of
WebTaskRunner::post(Delayed)CancellableTask.

This CL replaces CancellableTaskFactory with postDelayedCancellableTask.

BUG=663624
==========

to

==========
Worker: Replace CancellableTaskFactory with postDelayedCancellableTask

Before this CL, WorkerThread::ForceTerminationTask is implemented with
CancellableTaskFactory that is deprecated in favor of
WebTaskRunner::post(Delayed)CancellableTask.

This CL removes ForceTerminationTask/CancellableTaskFactory, and instead
implements the same functionality with postDelayedCancellableTask.

BUG=663624
==========

[nhiroki, 2016-11-10 07:25:18]

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

[commit-bot: I haz the power, 2016-11-10 07:25:30]

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

[nhiroki, 2016-11-10 07:26:38]

nhiroki@chromium.org changed reviewers:
+ falken@chromium.org, tzik@chromium.org

[nhiroki, 2016-11-10 07:26:39]

PTAL, thanks!

[commit-bot: I haz the power, 2016-11-10 07:27:32]

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

[commit-bot: I haz the power, 2016-11-10 07:27:33]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[tzik, 2016-11-10 07:34:28]

looks good

https://codereview.chromium.org/2490113002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/workers/WorkerThread.h (right):

https://codereview.chromium.org/2490113002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/workers/WorkerThread.h:301: TaskHandle
m_forcibleTerminationTaskHandle;
Class members need to be a complete type.
#include "platform/WebTaskRunner.h" ?

[nhiroki, 2016-11-10 07:39:26]

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

[commit-bot: I haz the power, 2016-11-10 07:39:39]

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

[nhiroki, 2016-11-10 07:39:52]

Thank you. Updated.

https://codereview.chromium.org/2490113002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/workers/WorkerThread.h (right):

https://codereview.chromium.org/2490113002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/workers/WorkerThread.h:301: TaskHandle
m_forcibleTerminationTaskHandle;
On 2016/11/10 07:34:27, tzik wrote:
> Class members need to be a complete type.
> #include "platform/WebTaskRunner.h" ?

Done.

[tzik, 2016-11-10 07:48:52]

lgtm

[commit-bot: I haz the power, 2016-11-10 09:18:59]

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

[commit-bot: I haz the power, 2016-11-10 09:19:00]

Dry run: This issue passed the CQ dry run.

[nhiroki, 2016-11-11 11:05:29]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2016-11-11 11:05:37]

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-11-11 14:35:50]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-11 14:38:22]

Description was changed from

==========
Worker: Replace CancellableTaskFactory with postDelayedCancellableTask

Before this CL, WorkerThread::ForceTerminationTask is implemented with
CancellableTaskFactory that is deprecated in favor of
WebTaskRunner::post(Delayed)CancellableTask.

This CL removes ForceTerminationTask/CancellableTaskFactory, and instead
implements the same functionality with postDelayedCancellableTask.

BUG=663624
==========

to

==========
Worker: Replace CancellableTaskFactory with postDelayedCancellableTask

Before this CL, WorkerThread::ForceTerminationTask is implemented with
CancellableTaskFactory that is deprecated in favor of
WebTaskRunner::post(Delayed)CancellableTask.

This CL removes ForceTerminationTask/CancellableTaskFactory, and instead
implements the same functionality with postDelayedCancellableTask.

BUG=663624

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

[commit-bot: I haz the power, 2016-11-11 14:38:23]

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