Use TaskScheduler instead of WorkerPool in sw_reporter_installer_win.cc.

Use TaskScheduler instead of WorkerPool in sw_reporter_installer_win.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).

No File IO (default):
  The task does not perform synchronous IO operations.

With Wait:
  The task waits 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/e32266ffc0f2401f27c9ac05fb0a881f899c619a
Cr-Commit-Position: refs/heads/master@{#435996}

[fdoray, 2016-12-01 21:49:42]

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:50:34]

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:21:59]

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

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

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[fdoray, 2016-12-02 13:40:14]

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

[commit-bot: I haz the power, 2016-12-02 13:40: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-02 14:28:05]

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

[commit-bot: I haz the power, 2016-12-02 14:28:06]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-12-02 19:24:16]

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

[fdoray, 2016-12-02 19:24:17]

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.

[Sorin Jianu, 2016-12-02 19:28:55]

lgtm

thank you!

[fdoray, 2016-12-02 19:30:45]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-12-02 19:31: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, 2016-12-02 19:39:21]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480707045972680,
"parent_rev": "d317b8cca1ed1701429fc74150dcabd5d483655c", "commit_rev":
"105900c3af42014204dc8c85fb0271f26d5e6d8b"}

[commit-bot: I haz the power, 2016-12-02 19:40:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-02 19:43:21]

Description was changed from

==========
Use TaskScheduler instead of WorkerPool in sw_reporter_installer_win.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).

No File IO (default):
  The task does not perform synchronous IO operations.

With Wait:
  The task waits 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 sw_reporter_installer_win.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).

No File IO (default):
  The task does not perform synchronous IO operations.

With Wait:
  The task waits 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/e32266ffc0f2401f27c9ac05fb0a881f899c619a
Cr-Commit-Position: refs/heads/master@{#435996}
==========

[commit-bot: I haz the power, 2016-12-02 19:43:22]

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