Use TaskScheduler instead of WorkerPool in google_brand_chromeos.cc.

Use TaskScheduler instead of WorkerPool in google_brand_chromeos.cc.

This CL replaces base::WorkerPool::PostTask() with
base::PostTaskWithTraits() call. The following traits are used:

Priority: BACKGROUND
  User won't notice if this task takes an arbitrarily long time
  to complete.

Shutdown behavior: CONTINUE_ON_SHUTDOWN
  Tasks posted with this mode which have not started executing before
  shutdown is initiated will never run. Tasks with this mode running at
  shutdown will be ignored (the worker will not be joined).

With file IO:
  The task is allowed to perform synchronous IO operations.

No Wait (default):
  The task does not wait on things other than synchronous file IO
  operations (e.g. WaitableEvent, ConditionVariable, join a thread,
  join a process, blocking system call that doesn't involve
  interactions with the file system).

BUG=659191
Committed: https://crrev.com/6c00462bd6aacdefa971db6a1d4c2354ccf913de
Cr-Commit-Position: refs/heads/master@{#436070}

[fdoray, 2016-12-01 21:55:07]

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

[commit-bot: I haz the power, 2016-12-01 21:55:32]

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, 2016-12-01 22:22:44]

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

[commit-bot: I haz the power, 2016-12-01 22:22:45]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-12-02 13:27:51]

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

[fdoray, 2016-12-02 13:27:51]

Please review this CL. In particular, make sure that the traits mentioned in the
CL description make sense for this task. Thanks!

Note: WorkerPool is being deprecated in favor of TaskScheduler. Read the design
doc
https://docs.google.com/document/d/1S2AAeoo1xa_vsLbDYBsDHCqhrkfiMgoIPlyRi6kxa...

Note: On shutdown, WorkerPool does not wait for tasks to run. To get the same
behavior in TaskScheduler, we used TaskShutdownBehavior::CONTINUE_ON_SHUTDOWN.

[fdoray, 2016-12-02 13:28:31]

fdoray@chromium.org changed reviewers:
+ pkasting@chromium.org
- jochen@chromium.org

[fdoray, 2016-12-02 13:28:32]

-jochen@
+pkasting@

Please review this CL. In particular, make sure that the traits mentioned in the
CL description make sense for this task. Thanks!


Note: WorkerPool is being deprecated in favor of TaskScheduler. Read the design
doc
https://docs.google.com/document/d/1S2AAeoo1xa_vsLbDYBsDHCqhrkfiMgoIPlyRi6kxa...


Note: On shutdown, WorkerPool does not wait for tasks to run. To get the same
behavior in TaskScheduler, we used TaskShutdownBehavior::CONTINUE_ON_SHUTDOWN.

[Peter Kasting, 2016-12-02 21:01:13]

LGTM

[fdoray, 2016-12-02 21:03:53]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-12-02 21:04:44]

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-12-02 23:08:45]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480712633318320, "parent_rev":
"aa9d2bdde4bfd57936cb6d5c5208d206a7c7556e", "commit_rev":
"baf36a6bc1763bc74ffd6645833f75dc944974d8"}

[commit-bot: I haz the power, 2016-12-02 23:09:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-02 23:13:17]

Description was changed from

==========
Use TaskScheduler instead of WorkerPool in google_brand_chromeos.cc.

This CL replaces base::WorkerPool::PostTask() with
base::PostTaskWithTraits() call. The following traits are used:

Priority: BACKGROUND
  User won't notice if this task takes an arbitrarily long time
  to complete.

Shutdown behavior: CONTINUE_ON_SHUTDOWN
  Tasks posted with this mode which have not started executing before
  shutdown is initiated will never run. Tasks with this mode running at
  shutdown will be ignored (the worker will not be joined).

With file IO:
  The task is allowed to perform synchronous IO operations.

No Wait (default):
  The task does not wait on things other than synchronous file IO
  operations (e.g. WaitableEvent, ConditionVariable, join a thread,
  join a process, blocking system call that doesn't involve
  interactions with the file system).

BUG=659191
==========

to

==========
Use TaskScheduler instead of WorkerPool in google_brand_chromeos.cc.

This CL replaces base::WorkerPool::PostTask() with
base::PostTaskWithTraits() call. The following traits are used:

Priority: BACKGROUND
  User won't notice if this task takes an arbitrarily long time
  to complete.

Shutdown behavior: CONTINUE_ON_SHUTDOWN
  Tasks posted with this mode which have not started executing before
  shutdown is initiated will never run. Tasks with this mode running at
  shutdown will be ignored (the worker will not be joined).

With file IO:
  The task is allowed to perform synchronous IO operations.

No Wait (default):
  The task does not wait on things other than synchronous file IO
  operations (e.g. WaitableEvent, ConditionVariable, join a thread,
  join a process, blocking system call that doesn't involve
  interactions with the file system).

BUG=659191

Committed: https://crrev.com/6c00462bd6aacdefa971db6a1d4c2354ccf913de
Cr-Commit-Position: refs/heads/master@{#436070}
==========

[commit-bot: I haz the power, 2016-12-02 23:13:18]

Patchset 1 (id:??) landed as
https://crrev.com/6c00462bd6aacdefa971db6a1d4c2354ccf913de
Cr-Commit-Position: refs/heads/master@{#436070}