Remove SequencedWorkerPool::PostDelayedWorkerTask().

Remove SequencedWorkerPool::PostDelayedWorkerTask().

There are no more callers of this method.

BUG=667892
Review-Url: https://codereview.chromium.org/2882523002
Cr-Commit-Position: refs/heads/master@{#472194}
Committed: https://chromium.googlesource.com/chromium/src/+/940082f0cb8fa26ab9d92603b9810a1de3ae4eac

[fdoray, 2017-05-11 18:09:10]

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

[fdoray, 2017-05-11 18:09:22]

fdoray@chromium.org changed reviewers:
+ gab@chromium.org

[fdoray, 2017-05-11 18:09:22]

PTAL

[commit-bot: I haz the power, 2017-05-11 18:10:07]

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, 2017-05-11 19:28:20]

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

[commit-bot: I haz the power, 2017-05-11 19:28:21]

Dry run: This issue passed the CQ dry run.

[gab, 2017-05-12 18:15:44]

Why not just remove it? What I meant was to make overrides private earlier in
cases where we successfully remove SWP callers of a TaskRunner override (and
thus can't just delete it).

[fdoray, 2017-05-16 13:36:21]

Description was changed from

==========
Make SequencedWorkerPool::PostDelayedWorkerTask() private.

There are no more callers of this method.

It is still possible to post a delayed task to a SequencedWorkerPool
via PostDelayedTask (inherited from TaskRunner). This matters because
a PostDelayedTask call doesn't prevent a SequencedWorkerPool member
to be substituted for a TaskRunner member.

BUG=667892
==========

to

==========
Remove SequencedWorkerPool::PostDelayedWorkerTask().

There are no more callers of this method.

BUG=667892
==========

[fdoray, 2017-05-16 13:38:04]

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

[fdoray, 2017-05-16 13:38:33]

PTAnL

[commit-bot: I haz the power, 2017-05-16 13:38:41]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[gab, 2017-05-16 15:18:42]

lgtm

https://codereview.chromium.org/2882523002/diff/20001/base/threading/sequence...
File base/threading/sequenced_worker_pool.cc (right):

https://codereview.chromium.org/2882523002/diff/20001/base/threading/sequence...
base/threading/sequenced_worker_pool.cc:1606: return inner_->PostTask(NULL,
SequenceToken(), shutdown_behavior, from_here,
nullptr

[fdoray, 2017-05-16 15:26:04]

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

[fdoray, 2017-05-16 15:26:09]

The CQ bit was unchecked by fdoray@chromium.org

[fdoray, 2017-05-16 15:26:12]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2017-05-16 15:26:12]

The patchset sent to the CQ was uploaded after l-g-t-m from gab@chromium.org
Link to the patchset: https://codereview.chromium.org/2882523002/#ps40001
(title: "CR-gab-12")

[fdoray, 2017-05-16 15:26:15]

https://codereview.chromium.org/2882523002/diff/20001/base/threading/sequence...
File base/threading/sequenced_worker_pool.cc (right):

https://codereview.chromium.org/2882523002/diff/20001/base/threading/sequence...
base/threading/sequenced_worker_pool.cc:1606: return inner_->PostTask(NULL,
SequenceToken(), shutdown_behavior, from_here,
On 2017/05/16 15:18:42, gab wrote:
> nullptr

Done.

[commit-bot: I haz the power, 2017-05-16 15:26:29]

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-05-16 17:19:18]

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

[commit-bot: I haz the power, 2017-05-16 17:19:19]

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

[fdoray, 2017-05-16 18:44:31]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-05-16 18:45: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-05-16 19:47:23]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494960271130040,
"parent_rev": "68d14c041a06c7a9acca3ec3113ce36324371d13", "commit_rev":
"940082f0cb8fa26ab9d92603b9810a1de3ae4eac"}

[commit-bot: I haz the power, 2017-05-16 19:47:38]

Description was changed from

==========
Remove SequencedWorkerPool::PostDelayedWorkerTask().

There are no more callers of this method.

BUG=667892
==========

to

==========
Remove SequencedWorkerPool::PostDelayedWorkerTask().

There are no more callers of this method.

BUG=667892

Review-Url: https://codereview.chromium.org/2882523002
Cr-Commit-Position: refs/heads/master@{#472194}
Committed:
https://chromium.googlesource.com/chromium/src/+/940082f0cb8fa26ab9d92603b981...
==========

[commit-bot: I haz the power, 2017-05-16 19:47:39]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/940082f0cb8fa26ab9d92603b981...