GitLab

Layout tests were migrated off RunLoop::QuitCurrentWhenIdleDeprecated()
to an explicit Shell::QuitMainMessageLoopForTesting() API by
https://chromium-review.googlesource.com/c/chromium/src/+/1106814

This changed the BlinkTestController's quit-main-message-loop semantics
such that multiple requests to quit the loop are now fatal, where
previously the second and subsequent requests would be silently
ignored, but left the potential for bugs due to async tasks quitting the
"current" loop rather than the one intended by the controller.

Fix the Shell API such that BlinkTestController quits the loop directly
rather than posting a quit task, and fix QuitMainMessageLoopForTesting()
to support being called multiple times.

Bug: 851416, 1051782
Change-Id: I3a9d3f78be2a22ceeef04611dc3a0cf0a25ee34c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2071860Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Peter Beverloo <peter@chromium.org>
Auto-Submit: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744392}