Issue #1442 & #1691 Follow-up - Part 22 - Changes that came with Dynamic Module Import. At the time we did not have ReadableStreams in our tree so these changes were left out. https://bugzilla.mozilla.org/show_bug.cgi?id=1499140 Implement support for dynamic module import in the interpreter.

js/src/builtin/Stream.cpp

@@ -284,15 +284,6 @@ ReportArgTypeError(JSContext* cx, const char* funName, const char* expectedType,
                                               funName, expectedType, bytes.get());
 }
 
-static MOZ_MUST_USE bool
-RejectWithPendingError(JSContext* cx, Handle<PromiseObject*> promise) {
-    // Not much we can do about uncatchable exceptions, just bail.
-    RootedValue exn(cx);
-    if (!GetAndClearException(cx, &exn))
-        return false;
-    return PromiseObject::reject(cx, promise, exn);
-}
-
 static MOZ_MUST_USE bool
 ReturnPromiseRejectedWithPendingError(JSContext* cx, const CallArgs& args)
 {
@@ -1183,7 +1174,7 @@ ReadableStreamTee_Cancel(JSContext* cx, Handle<TeeState*> teeState,
         // Step b: Let cancelResult be ! ReadableStreamCancel(stream, compositeReason).
         RootedObject cancelResult(cx, ReadableStream::cancel(cx, stream, compositeReasonVal));
         if (!cancelResult) {
-            if (!RejectWithPendingError(cx, promise))
+            if (!RejectPromiseWithPendingError(cx, promise))
                 return nullptr;
         } else {
             // Step c: Resolve teeState.[[promise]] with cancelResult.