[promises] Remove runtime call from fastpath in PromiseReject

src/runtime/runtime-promise.cc

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 #include "src/runtime/runtime-utils.h"
 
 #include "src/debug/debug.h"
 #include "src/elements.h"
 #include "src/promise-utils.h"
 
 namespace v8 {
@@ skipping 33 matching lines @@
     // as being a caught exception event.
     rejected_promise = isolate->GetPromiseOnStackOnThrow();
     isolate->debug()->OnAsyncTaskEvent(
         debug::kDebugEnqueuePromiseReject,
         isolate->debug()->NextAsyncTaskId(promise));
   }
   PromiseRejectEvent(isolate, promise, rejected_promise, value, true);
   return isolate->heap()->undefined_value();
 }
 
+RUNTIME_FUNCTION(Runtime_ReportPromiseReject) {
+  DCHECK_EQ(2, args.length());
+  HandleScope scope(isolate);
+  CONVERT_ARG_HANDLE_CHECKED(JSPromise, promise, 0);
+  CONVERT_ARG_HANDLE_CHECKED(Object, value, 1);
+  isolate->ReportPromiseReject(Handle<JSObject>::cast(promise), value,
+                               v8::kPromiseRejectWithNoHandler);
+  return isolate->heap()->undefined_value();
+}
+
 RUNTIME_FUNCTION(Runtime_PromiseRevokeReject) {
   DCHECK_EQ(1, args.length());
   HandleScope scope(isolate);
   CONVERT_ARG_HANDLE_CHECKED(JSPromise, promise, 0);
   // At this point, no revocation has been issued before
   CHECK(!promise->has_handler());
   isolate->ReportPromiseReject(promise, Handle<Object>(),
                                v8::kPromiseHandlerAddedAfterReject);
   return isolate->heap()->undefined_value();
 }
@@ skipping 59 matching lines @@
 void EnqueuePromiseReactionJob(Isolate* isolate, Handle<JSPromise> promise,
                                Handle<PromiseReactionJobInfo> info,
                                int status) {
   if (isolate->debug()->is_active()) {
     SetDebugInfo(isolate, promise, info, status);
   }
 
   isolate->EnqueueMicrotask(info);
 }
 
-void PromiseSet(Isolate* isolate, Handle<JSPromise> promise, int status,
-                Handle<Object> result) {
-  promise->set_status(status);
-  promise->set_result(*result);
-  promise->set_deferred_promise(isolate->heap()->undefined_value());
-  promise->set_deferred_on_resolve(isolate->heap()->undefined_value());
-  promise->set_deferred_on_reject(isolate->heap()->undefined_value());
-  promise->set_fulfill_reactions(isolate->heap()->undefined_value());
-  promise->set_reject_reactions(isolate->heap()->undefined_value());
-}
-
-void PromiseFulfill(Isolate* isolate, Handle<JSPromise> promise, int status,
-                    Handle<Object> value) {
-  if (isolate->debug()->is_active()) {
-    isolate->debug()->OnAsyncTaskEvent(
-        status == v8::Promise::kFulfilled ? debug::kDebugEnqueuePromiseResolve
-                                          : debug::kDebugEnqueuePromiseReject,
-        isolate->debug()->NextAsyncTaskId(promise));
-  }
-  // Check if there are any callbacks.
-  if (!promise->deferred_promise()->IsUndefined(isolate)) {
-    Handle<Object> tasks((status == v8::Promise::kFulfilled)
-                             ? promise->fulfill_reactions()
-                             : promise->reject_reactions(),
-                         isolate);
-    Handle<PromiseReactionJobInfo> info =
-        isolate->factory()->NewPromiseReactionJobInfo(
-            value, tasks, handle(promise->deferred_promise(), isolate),
-            handle(promise->deferred_on_resolve(), isolate),
-            handle(promise->deferred_on_reject(), isolate),
-            isolate->native_context());
-    EnqueuePromiseReactionJob(isolate, promise, info, status);
-  }
-
-  PromiseSet(isolate, promise, status, value);
-}
-
 }  // namespace
 
-RUNTIME_FUNCTION(Runtime_PromiseReject) {
-  DCHECK_EQ(3, args.length());
-  HandleScope scope(isolate);
-  CONVERT_ARG_HANDLE_CHECKED(JSPromise, promise, 0);
-  CONVERT_ARG_HANDLE_CHECKED(Object, reason, 1);
-  CONVERT_BOOLEAN_ARG_CHECKED(debug_event, 2);
-
-  PromiseRejectEvent(isolate, promise, promise, reason, debug_event);
-  PromiseFulfill(isolate, promise, v8::Promise::kRejected, reason);
-
-  return isolate->heap()->undefined_value();
-}
-
 RUNTIME_FUNCTION(Runtime_EnqueuePromiseReactionJob) {
   HandleScope scope(isolate);
   DCHECK_EQ(3, args.length());
   CONVERT_ARG_HANDLE_CHECKED(JSPromise, promise, 0);
   CONVERT_ARG_HANDLE_CHECKED(PromiseReactionJobInfo, info, 1);
   CONVERT_SMI_ARG_CHECKED(status, 2);
   EnqueuePromiseReactionJob(isolate, promise, info, status);
   return isolate->heap()->undefined_value();
 }
 
@@ skipping 84 matching lines @@
   HandleScope scope(isolate);
   DCHECK_EQ(1, args.length());
   CONVERT_ARG_HANDLE_CHECKED(JSPromise, promise, 0);
   isolate->RunPromiseHook(PromiseHookType::kAfter, promise,
                           isolate->factory()->undefined_value());
   return isolate->heap()->undefined_value();
 }
 
 }  // namespace internal
 }  // namespace v8