Don't immediately type-check when converting a v8::Value into a Scrip…

…tPromise<>

Typed promises in the generated bindings were introduced in
https://chromium-review.googlesource.com/c/chromium/src/+/5381805.

That CL converted to a rejected promise when the input value wasn't a
promise and was the wrong type. This reverts to the old behavior of
unconditionally wrapping the value in a resolved promise, and
handling type checking in the reaction.

Change-Id: I6c875be415bf62f2e7fdb9d1150f8deae5be0032
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5908893
Reviewed-by: Maks Orlovich <[email protected]>
Reviewed-by: Andrey Kosyakov <[email protected]>
Auto-Submit: Nate Chapin <[email protected]>
Commit-Queue: Maks Orlovich <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1364916}

fledge/tentative/auction-config.https.window.js

@@ -475,10 +475,7 @@ makeTest({
 makeTest({
   name: 'perBuyerCurrencies not convertible to dictionary',
   expect: EXPECT_PROMISE_ERROR,
-  // Because this is not convertible by IDL to the proper dictionary type, this
-  // will immediately be converted to a rejected promise inside of the
-  // runBasicFledgeAuction() invocation, and that await will handle the
-  // rejection. Therefore, no expectPromiseError.
+  expectPromiseError: EXPECT_EXCEPTION(TypeError),
   auctionConfigOverrides: {perBuyerCurrencies: 123}
 });