Remove SchedulerWorkerPoolImpl::ReEnqueueSequenceCallback.

Remove SchedulerWorkerPoolImpl::ReEnqueueSequenceCallback.

This was useful when we thought that sequences could contain
tasks with different traits and be enqueued in different priority
queues during their lifetime. This is no longer a supported use case.

BUG=553459
Review-Url: https://codereview.chromium.org/2862143004
Cr-Commit-Position: refs/heads/master@{#470034}
Committed: https://chromium.googlesource.com/chromium/src/+/be1c345dc5a61438d6aeb57544689cfe22598ffc

[fdoray, 2017-05-05 19:28:45]

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

[fdoray, 2017-05-05 19:28:46]

PTAL

[robliao, 2017-05-05 21:23:39]

lgtm

https://codereview.chromium.org/2862143004/diff/1/base/task_scheduler/task_sc...
File base/task_scheduler/task_scheduler_impl.h (right):

https://codereview.chromium.org/2862143004/diff/1/base/task_scheduler/task_sc...
base/task_scheduler/task_scheduler_impl.h:21: #include
"base/task_scheduler/sequence.h"
This can probably be moved into the cc file.

[fdoray, 2017-05-08 14:54:32]

https://codereview.chromium.org/2862143004/diff/1/base/task_scheduler/task_sc...
File base/task_scheduler/task_scheduler_impl.h (right):

https://codereview.chromium.org/2862143004/diff/1/base/task_scheduler/task_sc...
base/task_scheduler/task_scheduler_impl.h:21: #include
"base/task_scheduler/sequence.h"
On 2017/05/05 21:23:39, robliao wrote:
> This can probably be moved into the cc file.

Done.

[fdoray, 2017-05-08 14:55:13]

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

[commit-bot: I haz the power, 2017-05-08 14:55:37]

Dry run: CQ is trying da patch.

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

[fdoray, 2017-05-08 14:56:32]

The CQ bit was unchecked by fdoray@chromium.org

[fdoray, 2017-05-08 14:56:36]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2017-05-08 14:56:36]

The patchset sent to the CQ was uploaded after l-g-t-m from robliao@chromium.org
Link to the patchset: https://codereview.chromium.org/2862143004/#ps20001
(title: "CR-robliao-remove-includes")

[commit-bot: I haz the power, 2017-05-08 14:56:51]

CQ is trying da patch.

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

[gab, 2017-05-08 15:55:51]

lgtm

[commit-bot: I haz the power, 2017-05-08 17:31:03]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494255396109080,
"parent_rev": "94bde17fed2469a58c0fa6ae4374d71264ef1370", "commit_rev":
"be1c345dc5a61438d6aeb57544689cfe22598ffc"}

[commit-bot: I haz the power, 2017-05-08 17:31:15]

Description was changed from

==========
Remove SchedulerWorkerPoolImpl::ReEnqueueSequenceCallback.

This was useful when we thought that sequences could contain
tasks with different traits and be enqueued in different priority
queues during their lifetime. This is no longer a supported use case.

BUG=553459
==========

to

==========
Remove SchedulerWorkerPoolImpl::ReEnqueueSequenceCallback.

This was useful when we thought that sequences could contain
tasks with different traits and be enqueued in different priority
queues during their lifetime. This is no longer a supported use case.

BUG=553459

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

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

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