DescriptionFix UnlockEndpointIsDelayed again.
On DrMemory and Valgrind builds, it takes more than 1ms just to post the task
inside UnlockEndpoint() and then enter the message loop, so the delayed task was
being executed inside RunUntilIdle().
Remove the call to RunUntilIdle() and the check that the callback wasn't called
inside it. The check at 314 is sufficient to verify that at least 1ms has
passed, which in normal compiles means that a delay was applied.
On DrMemory and Valgrind builds, the test may just verify that everything is
running very slowly, but at least it will be deterministic.
BUG=451999
TEST=net_unittests
Committed: https://crrev.com/ba6836202b8d60478d96fd8509ad98cd89e1ee1c
Cr-Commit-Position: refs/heads/master@{#314093}
Patch Set 1 #
Messages
Total messages: 11 (3 generated)
|