[promises] Move PromiseHasUserDefinedRejectHandler to c++

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 d8e3dcaad3ee1fb21eab756a55a66d3e938739dd..a2c5acb5c8b5affc4b93ff90934304c2b0defead 100644
--- a/src/runtime/runtime-promise.cc
+++ b/src/runtime/runtime-promise.cc
@@ -279,7 +279,7 @@ RUNTIME_FUNCTION(Runtime_PromiseRejectReactions) {
 }
 
 RUNTIME_FUNCTION(Runtime_PromiseMarkAsHandled) {
-  HandleScope scope(isolate);
+  SealHandleScope shs(isolate);
   DCHECK(args.length() == 1);
   CONVERT_ARG_HANDLE_CHECKED(JSPromise, promise, 0);

[adamk, 2016/12/27 17:44:58]

When using SealHandleScope you don't want to create handles, so this should be

CONVERT_ARG_CHECKED

instead

 
@@ -287,6 +287,15 @@ RUNTIME_FUNCTION(Runtime_PromiseMarkAsHandled) {
   return isolate->heap()->undefined_value();
 }
 
+RUNTIME_FUNCTION(Runtime_PromiseMarkHandledHint) {
+  SealHandleScope shs(isolate);
+  DCHECK(args.length() == 1);
+  CONVERT_ARG_HANDLE_CHECKED(JSPromise, promise, 0);

[adamk, 2016/12/27 17:44:58]

Same down here.

+
+  promise->set_handled_hint(true);
+  return isolate->heap()->undefined_value();
+}
+
 RUNTIME_FUNCTION(Runtime_PromiseHookInit) {
   HandleScope scope(isolate);
   DCHECK_EQ(2, args.length());