Reland "[inspector] Ensure progress of default task runners in inspector tests"

This is a reland of a7eff8cb

Sheriffs: This CL effectively enables GC for inspector tests. It may
uncover bugs in the existing inspector tests. If that happens please
disable the failing test in test/inspector/inspector.status instead of
reverting this CL. See https://crbug.com/v8/10748.

Original change's description:
> [inspector] Ensure progress of default task runners in inspector tests
>
> Bug: v8:10747, chromium:1098187
> Change-Id: I0215d2d85db4dd1d7fa376a0982cda5658f1683d
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2315982
> Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
> Reviewed-by: Peter Marshall <petermarshall@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#69387}

Bug: v8:10747
Bug: chromium:1098187
Change-Id: Iab8f10b0535b0f35948fb507537f2bf65af5f33a
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2354815
Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
Reviewed-by: Peter Marshall <petermarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#69398}

test/inspector/inspector.status

@@ -97,4 +97,29 @@
   'cpu-profiler/console-profile-wasm': [SKIP],
 }],  # 'tsan == True'
 
+##############################################################################
+['gc_stress or gc_fuzzer or variant == stress_incremental_marking', {
+  # Skip tests that fail with GC stress: https://crbug.com/v8/10748
+  'debugger/get-possible-breakpoints': [SKIP],
+  'debugger/get-possible-breakpoints-array-literal': [SKIP],
+  'debugger/get-possible-breakpoints-master': [SKIP],
+  'debugger/regression-424142': [SKIP],
+  'debugger/return-break-locations': [SKIP],
+  'debugger/set-breakpoint-at-last-line': [SKIP],
+  'debugger/set-breakpoint-breaks-on-first-breakable-location': [SKIP],
+  'runtime-call-stats/collection': [SKIP],
+  'runtime/internal-properties-entries': [SKIP],
+  'type-profiler/type-profile-start-stop': [SKIP],
+}], # gc_stress
+
+##############################################################################
+['variant == stress_js_bg_compile_wasm_code_gc', {
+  # Skip tests that fail with GC stress: https://crbug.com/v8/10748
+  'debugger/wasm-debug-command': [SKIP],
+  'debugger/wasm-global-names': [SKIP],
+  'debugger/wasm-set-breakpoint-liftoff': [SKIP],
+  'debugger/wasm-source': [SKIP],
+  'debugger/wasm-stepping-with-skiplist': [SKIP],
+}], # stress_js_bg_compile_wasm_code_gc
+
 ]

test/inspector/task-runner.cc

@@ -4,6 +4,9 @@
 
 #include "test/inspector/task-runner.h"
 
+#include "include/libplatform/libplatform.h"
+#include "src/flags/flags.h"
+
 #if !defined(_WIN32) && !defined(_WIN64)
 #include <unistd.h>  // NOLINT
 #endif               // !defined(_WIN32) && !defined(_WIN64)
@@ -75,6 +78,15 @@ void TaskRunner::RunMessageLoop(bool only_protocol) {
       task->Run(data_.get());
       delete task;
     }
+    // Also pump isolate's foreground task queue to ensure progress.
+    // This can be removed once https://crbug.com/v8/10747 is fixed.
+    // TODO(10748): Enable --stress-incremental-marking after the existing
+    // tests are fixed.
+    if (!i::FLAG_stress_incremental_marking) {
+      while (v8::platform::PumpMessageLoop(
+          v8::internal::V8::GetCurrentPlatform(), isolate())) {
+      }
+    }
   }
 }