GitLab

The previous attempt to land this CL was reverted because a value that
we didn't expect was returned by ::GetThreadPriority() on Win7 bots.
This CL adds logging so we can find out what value is returned and add
it to the expected values if need be.

Previous attempts to land:
  https://chromium-review.googlesource.com/1171482
  https://chromium-review.googlesource.com/1185529

NOTE: If this causes new test failures, we'll add the value returned
by GetThreadPriority() to the switch/case of allowed values.

This CL adds a feature to use THREAD_MODE_BACKGROUND_BEGIN instead of
THREAD_PRIORITY_LOWEST for ThreadPriority::BACKGROUND threads. This
lowers the disk and network I/O priority of the thread in addition to
the CPU scheduling priority. MSDN recommends using this setting for
threads that perform background work.
https://docs.microsoft.com/en-us/windows/desktop/api/processthreadsapi/nf-processthreadsapi-setthreadpriority

TBR=gab@chromium.org

Bug: 872820
Change-Id: I54de2f17c66985ac437ca7809d3b515b22c72544
Reviewed-on: https://chromium-review.googlesource.com/1190842
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#586311}