Bug 1821563 - Do not create a promise from an uncatchable exception r=mgaudet

Differential Revision: https://phabricator.services.mozilla.com/D172654
2023-03-15 14:38:05 +00:00 · 2023-03-15 14:38:05 +00:00 · 35fe8e091d
--- a/dom/streams/StreamUtils.h
+++ b/dom/streams/StreamUtils.h
@ -35,6 +35,10 @@ MOZ_CAN_RUN_SCRIPT static already_AddRefed<Promise> PromisifyAlgorithm(
    aFunc(aRv);
  }

+  if (aRv.IsUncatchableException()) {
+    return nullptr;
+  }
+
  if (aRv.Failed()) {
    return Promise::CreateRejectedWithErrorResult(aGlobal, aRv);
  }
--- a/testing/web-platform/tests/encoding/streams/stringification-crash.html
+++ b/testing/web-platform/tests/encoding/streams/stringification-crash.html
@ -0,0 +1,20 @@
+<!DOCTYPE html>
+<html class="test-wait">
+<script id="workerScript" type="text/worker">
+  // Blob will pass a huge Uint8Array object whose stringification will take long
+  const encoder = new TextEncoderStream()
+  const blob = new Blob([new ArrayBuffer(1338720)])
+  const writable = new WritableStream()
+  blob.stream().pipeThrough(encoder).pipeTo(writable)
+  // Wait a bit for the piping to start
+  Promise.resolve().then(() => postMessage('started'));
+</script>
+<script>
+  const worker = new Worker(URL.createObjectURL(new Blob([
+    document.getElementById("workerScript").textContent
+  ])))
+  worker.onmessage = () => {
+    worker.terminate();
+    document.documentElement.classList.remove("test-wait");
+  };
+</script>