[promises] Remove one runtime call to create_resolving_functions

src/runtime/runtime-promise.cc

Index: src/runtime/runtime-promise.cc
diff --git a/src/runtime/runtime-promise.cc b/src/runtime/runtime-promise.cc
index a6d3ff93d403c2b6c5a609c9bbbc160bf124e442..540c6131467d7b09717553779ad06025b2242c7f 100644
--- a/src/runtime/runtime-promise.cc
+++ b/src/runtime/runtime-promise.cc
@@ -6,6 +6,7 @@
 
 #include "src/debug/debug.h"
 #include "src/elements.h"
+#include "src/promise-utils.h"
 
 namespace v8 {
 namespace internal {
@@ -139,13 +140,20 @@ RUNTIME_FUNCTION(Runtime_EnqueuePromiseReactionJob) {
 
 RUNTIME_FUNCTION(Runtime_EnqueuePromiseResolveThenableJob) {
   HandleScope scope(isolate);
-  DCHECK(args.length() == 4);
-  CONVERT_ARG_HANDLE_CHECKED(JSReceiver, resolution, 0);
-  CONVERT_ARG_HANDLE_CHECKED(JSReceiver, then, 1);
-  CONVERT_ARG_HANDLE_CHECKED(JSFunction, resolve, 2);
-  CONVERT_ARG_HANDLE_CHECKED(JSFunction, reject, 3);
-  Handle<Object> debug_id;
-  Handle<Object> debug_name;
+  DCHECK(args.length() == 3);
+  CONVERT_ARG_HANDLE_CHECKED(JSObject, promise, 0);
+  CONVERT_ARG_HANDLE_CHECKED(JSReceiver, resolution, 1);
+  CONVERT_ARG_HANDLE_CHECKED(JSReceiver, then, 2);
+
+  // TODO(gsathya): Add fast path for native promises with unmodified
+  // PromiseThen (which don't need these resolving functions, but
+  // instead can just call resolve/reject directly).
+  Handle<JSFunction> resolve, reject;
+  PromiseUtils::CreateResolvingFunctions(isolate, promise,
+                                         isolate->factory()->ToBoolean(false),

[adamk, 2016/11/10 18:10:55]

isolate->factory()->false_value()

[gsathya, 2016/11/11 02:34:38]

Done.

+                                         &resolve, &reject);
+
+  Handle<Object> debug_id, debug_name;
   if (isolate->debug()->is_active()) {
     debug_id =
         handle(Smi::FromInt(isolate->GetNextDebugMicrotaskId()), isolate);
@@ -157,10 +165,13 @@ RUNTIME_FUNCTION(Runtime_EnqueuePromiseResolveThenableJob) {
     debug_id = isolate->factory()->undefined_value();
     debug_name = isolate->factory()->undefined_value();
   }
+
   Handle<PromiseResolveThenableJobInfo> info =
       isolate->factory()->NewPromiseResolveThenableJobInfo(
-          resolution, then, resolve, reject, debug_id, debug_name);
+          resolution, then, resolve, reject, debug_id, debug_name,
+          isolate->native_context());
   isolate->EnqueueMicrotask(info);
+
   return isolate->heap()->undefined_value();
 }