Use TaskScheduler instead of blocking pool in session_manager_operation.cc.

Use TaskScheduler instead of blocking pool in session_manager_operation.cc.

The blocking pool is being deprecated in favor of TaskScheduler.

BUG=667892
R=bartfab@chromium.org

Review-Url: https://codereview.chromium.org/2836133002
Cr-Commit-Position: refs/heads/master@{#467662}
Committed: https://chromium.googlesource.com/chromium/src/+/4b1f1b0c63b6b65673c020d1f2b1177ce6619471

[fdoray, 2017-04-24 19:29:17]

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

[fdoray, 2017-04-24 19:29:19]

PTAL!

This CL was generated automatically.

Please make sure that the appropriate TaskTraits are used to post tasks.
https://cs.chromium.org/chromium/src/base/task_scheduler/task_traits.h
If everything looks good, please lgtm and CQ this CL. Otherwise, tell
us what's wrong.

Thanks!

- FAQ -

What if bots are red? Ignore this CL. A human will look at errors shortly.

What if the task priority is not set explicitly (no .WithTaskPriority())?
When there is no explicit priority, the priority is inherited from the
calling context (e.g. a task posted from a BACKGROUND task without an
explicit priority will have a BACKGROUND priority).

What if the shutdown behavior is not set explicitly
(no .WithShutdownBehavior())?
If shutdown behavior is important for a task, it should be set explicitly.
It's not necessary to specify it if you're fine with either BLOCK_SHUTDOWN
or SKIP_ON_SHUTDOWN.

[commit-bot: I haz the power, 2017-04-24 19:30:06]

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-04-24 20:00:15]

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

[commit-bot: I haz the power, 2017-04-24 20:00:16]

Dry run: This issue passed the CQ dry run.

[fdoray, 2017-04-26 18:31:20]

Description was changed from

==========
Use TaskScheduler instead of blocking pool in session_manager_operation.cc.

The blocking pool is being deprecated in favor of TaskScheduler.

BUG=667892
R=bartfab@chromium.org
==========

to

==========
Use TaskScheduler instead of blocking pool in session_manager_operation.cc.

The blocking pool is being deprecated in favor of TaskScheduler.

BUG=667892
R=bartfab@chromium.org
==========

[fdoray, 2017-04-26 18:31:20]

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

[fdoray, 2017-04-26 18:31:49]

+mnissler@chromium.org since bartfab is "(slow)".

On 2017/04/24 19:29:19, fdoray wrote:
> PTAL!
> 
> This CL was generated automatically.
> 
> Please make sure that the appropriate TaskTraits are used to post tasks.
> https://cs.chromium.org/chromium/src/base/task_scheduler/task_traits.h
> If everything looks good, please lgtm and CQ this CL. Otherwise, tell
> us what's wrong.
> 
> Thanks!
> 
> - FAQ -
> 
> What if bots are red? Ignore this CL. A human will look at errors shortly.
> 
> What if the task priority is not set explicitly (no .WithTaskPriority())?
> When there is no explicit priority, the priority is inherited from the
> calling context (e.g. a task posted from a BACKGROUND task without an
> explicit priority will have a BACKGROUND priority).
> 
> What if the shutdown behavior is not set explicitly
> (no .WithShutdownBehavior())?
> If shutdown behavior is important for a task, it should be set explicitly.
> It's not necessary to specify it if you're fine with either BLOCK_SHUTDOWN
> or SKIP_ON_SHUTDOWN.

[Mattias Nissler (ping if slow), 2017-04-26 20:43:22]

LGTM!

[fdoray, 2017-04-27 12:13:51]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-04-27 12:14:06]

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-04-27 13:58:03]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493295231668070, "parent_rev":
"95bfc8dbca7ee83b4d8f3f352f7b6862ca31c559", "commit_rev":
"4b1f1b0c63b6b65673c020d1f2b1177ce6619471"}

[commit-bot: I haz the power, 2017-04-27 13:58:21]

Description was changed from

==========
Use TaskScheduler instead of blocking pool in session_manager_operation.cc.

The blocking pool is being deprecated in favor of TaskScheduler.

BUG=667892
R=bartfab@chromium.org
==========

to

==========
Use TaskScheduler instead of blocking pool in session_manager_operation.cc.

The blocking pool is being deprecated in favor of TaskScheduler.

BUG=667892
R=bartfab@chromium.org

Review-Url: https://codereview.chromium.org/2836133002
Cr-Commit-Position: refs/heads/master@{#467662}
Committed:
https://chromium.googlesource.com/chromium/src/+/4b1f1b0c63b6b65673c020d1f2b1...
==========

[commit-bot: I haz the power, 2017-04-27 13:58:22]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/4b1f1b0c63b6b65673c020d1f2b1...