tree ee9c1b82192acdea138016d2ba4ffb8348551bd5
parent 054c9865a0fe4074122022bfd4458fc08305a1d5
author David 'Digit' Turner <digit@google.com> 1790696947 +0200
committer gn-scoped@luci-project-accounts.iam.gserviceaccount.com <gn-scoped@luci-project-accounts.iam.gserviceaccount.com> 1790843418 -0700

Fix minor Scheduler data races.

Ensure that Scheduler::{is_failed_,has_been_shutdown_}
are properly guarded by a lock since they are accessed
from multiple threads concurrently.

In Scheduler::FailWithError(), ensure the PostTask()
call happens under the lock.

In particular, this can happen when several background worker
threads are posting tasks to the queue, and one of them calls
Scheduler::FailWithError() which will implicitly call PostQuit().

Change-Id: Ie0b70be0838b9fdcf4f41c3ae53bc5bd8e6d71d3
Reviewed-on: https://gn-review.googlesource.com/c/gn/+/27080
Commit-Queue: David Turner <digit@google.com>
Reviewed-by: Takuto Ikuta <tikuta@google.com>