Mark SequencedWorkerPool::GetWorkerPoolForCurrentThread() as deprecated.

Mark SequencedWorkerPool::GetWorkerPoolForCurrentThread() as deprecated.

This methods does not work when SequencedWorkerPool is redirected to
TaskScheduler (always returns nullptr).

The only remaining use is in base/threading/sequenced_task_runner_handle.cc
(which uses another way to get the current SequencedTaskRunner when
redirection to TaskScheduler is enabled).

BUG=622400
Committed: https://crrev.com/3be8af75b2f307116aabebc05dd92b5169145f58
Cr-Commit-Position: refs/heads/master@{#422238}

[fdoray, 2016-09-30 19:04:54]

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

[fdoray, 2016-09-30 19:04:55]

PTAL

[danakj, 2016-09-30 19:14:55]

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

https://codereview.chromium.org/2383203002/diff/20001/base/threading/sequence...
base/threading/sequenced_worker_pool.h:173: // DEPRECATED. The only remaining
use is in
Can you say what people should use instead?

[gab, 2016-09-30 19:24:01]

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

https://codereview.chromium.org/2383203002/diff/20001/base/threading/sequence...
base/threading/sequenced_worker_pool.h:173: // DEPRECATED. The only remaining
use is in
On 2016/09/30 19:14:55, danakj wrote:
> Can you say what people should use instead?

IMO something like:

// DEPRECATED. Use SequencedTaskRunnerHandle::Get() instead. Consequentially the
only remaining use case is in sequenced_task_runner_handle.cc to implement that
and will soon be removed along with SequencedWorkerPool :
http://crbug.com/622400.

[fdoray, 2016-09-30 19:28:55]

PTAnL

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

https://codereview.chromium.org/2383203002/diff/20001/base/threading/sequence...
base/threading/sequenced_worker_pool.h:173: // DEPRECATED. The only remaining
use is in
On 2016/09/30 19:24:01, gab wrote:
> On 2016/09/30 19:14:55, danakj wrote:
> > Can you say what people should use instead?
> 
> IMO something like:
> 
> // DEPRECATED. Use SequencedTaskRunnerHandle::Get() instead. Consequentially
the
> only remaining use case is in sequenced_task_runner_handle.cc to implement
that
> and will soon be removed along with SequencedWorkerPool :
> http://crbug.com/622400.

Done.

[gab, 2016-09-30 19:30:33]

lgtm

[danakj, 2016-09-30 20:19:20]

LGTM

[fdoray, 2016-09-30 20:25:05]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-09-30 20:25:35]

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-09-30 22:15:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-30 22:17:47]

Description was changed from

==========
Mark SequencedWorkerPool::GetWorkerPoolForCurrentThread() as deprecated.

This methods does not work when SequencedWorkerPool is redirected to
TaskScheduler (always returns nullptr).

The only remaining use is in base/threading/sequenced_task_runner_handle.cc
(which uses another way to get the current SequencedTaskRunner when
redirection to TaskScheduler is enabled).

BUG=622400
==========

to

==========
Mark SequencedWorkerPool::GetWorkerPoolForCurrentThread() as deprecated.

This methods does not work when SequencedWorkerPool is redirected to
TaskScheduler (always returns nullptr).

The only remaining use is in base/threading/sequenced_task_runner_handle.cc
(which uses another way to get the current SequencedTaskRunner when
redirection to TaskScheduler is enabled).

BUG=622400

Committed: https://crrev.com/3be8af75b2f307116aabebc05dd92b5169145f58
Cr-Commit-Position: refs/heads/master@{#422238}
==========

[commit-bot: I haz the power, 2016-09-30 22:17:48]

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