Check TryCatch::CanContinue() in module scripts

Bug: 1129793, 1114601
Change-Id: Ifc26c2c9e75b25d4ff001395b4faf072c41974c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2485411Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827929}

third_party/blink/renderer/bindings/core/v8/v8_script_runner.cc

@@ -727,11 +727,20 @@ ScriptEvaluationResult V8ScriptRunner::EvaluateModule(
     // Do not perform a microtask checkpoint here. A checkpoint is performed
     // only after module error handling to ensure proper timing with and
     // without top-level await.
+
+    v8::MaybeLocal<v8::Value> maybe_result =
+        record->Evaluate(script_state->GetContext());
+
+    if (!try_catch.CanContinue())
+      return ScriptEvaluationResult::FromModuleAborted();
+
     v8::Local<v8::Value> v8_result;
-    if (!record->Evaluate(script_state->GetContext()).ToLocal(&v8_result)) {
+    if (!maybe_result.ToLocal(&v8_result)) {
+      DCHECK(try_catch.HasCaught());
       result =
           ScriptEvaluationResult::FromModuleException(try_catch.Exception());
     } else {
+      DCHECK(!try_catch.HasCaught());
       result = ScriptEvaluationResult::FromModuleSuccess(v8_result);
     }
 

third_party/blink/web_tests/external/wpt/workers/Worker-terminate-forever-during-evaluation-expected.txt

-This is a testharness.js-based test.
-PASS Worker is terminated during top-level script evaluation
-FAIL Worker is terminated during top-level script evaluation (module) assert_unreached: onerror Reached unreachable code
-PASS Worker is terminated during importScripts() call
-PASS Worker is terminated during nested importScripts() call
-PASS Worker is terminated during dynamic import()
-PASS Worker is terminated during dynamic import() (module)
-Harness: the test ran to completion.
-