Relax conditions in compiler dispatcher test

Just ensure that all background and foreground tasks finished, then we
should be in a defined state

BUG=v8:6069
R=rmcilroy@chromium.org

Change-Id: Ie5bd11c61402dccb2c65cb8fe57fd1c0f550e9a7
Reviewed-on: https://chromium-review.googlesource.com/456418Reviewed-by: Ross McIlroy <rmcilroy@chromium.org>
Commit-Queue: Jochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#43926}

test/unittests/compiler-dispatcher/compiler-dispatcher-unittest.cc

@@ -722,10 +722,11 @@ TEST_F(CompilerDispatcherTest, FinishNowDuringAbortAll) {
   // Force the compilation to finish, even while aborting.
   ASSERT_TRUE(dispatcher.FinishNow(shared));
   ASSERT_TRUE(dispatcher.jobs_.empty());
+
+  // Busy wait for the background task to finish.
   for (;;) {
     base::LockGuard<base::Mutex> lock(&dispatcher.mutex_);
     if (dispatcher.num_background_tasks_ == 0) {
-      ASSERT_FALSE(dispatcher.abort_);
       break;
     }
   }
@@ -733,6 +734,13 @@ TEST_F(CompilerDispatcherTest, FinishNowDuringAbortAll) {
   ASSERT_TRUE(platform.ForegroundTasksPending());
   ASSERT_TRUE(platform.IdleTaskPending());
   ASSERT_FALSE(platform.BackgroundTasksPending());
+
+  platform.RunForegroundTasks();
+  {
+    base::LockGuard<base::Mutex> lock(&dispatcher.mutex_);
+    ASSERT_FALSE(dispatcher.abort_);
+  }
+
   platform.ClearForegroundTasks();
   platform.ClearIdleTask();
 }