tree 07bec93a05a64eccc72dc8489b9625c3527c84be
parent 4b460a85426ead4865410a7a1f62249c7781dee4
author Robert Sesek <rsesek@chromium.org> 1499276294 +0900
committer Qijiang Fan <fqj@google.com> 1591306504 +0900

Do not Reset the waitable events in TraceEventTestFixture.ThreadOnceBlocking.

The events are created as automatic-reset, so it is not necessary to reset them
manually. Doing so for the |task_start_event| and |task_stop_event| is racy,
since the test main thread could signal and then immediately reset the event.
If the second thread does not make progress between those two calls, then it
will miss observing the signal and the test will hang forever.

Bug: 681167
Change-Id: Ia65eccd5fbbb83af53f4501ce4abb99ed3c9131c
Reviewed-on: https://chromium-review.googlesource.com/559510
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Commit-Queue: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#484301}

CrOS-Libchrome-Original-Commit: da4be990616dee7746ab65aab2026c10a61d1586