[promises] Remove runtime call from fastpath in PromiseReject

src/js/prologue.js

 // Copyright 2015 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.
 
 (function(global, utils, extrasUtils) {
 
 "use strict";
 
 %CheckIsBootstrapping();
 
@@ skipping 248 matching lines @@
 utils.OverrideFunction = OverrideFunction;
 utils.SetUpLockedPrototype = SetUpLockedPrototype;
 utils.PostNatives = PostNatives;
 utils.PostExperimentals = PostExperimentals;
 utils.PostDebug = PostDebug;
 
 %ToFastProperties(utils);
 
 // -----------------------------------------------------------------------
 
-%OptimizeObjectForAddingMultipleProperties(extrasUtils, 5);
+function DoRejectPromise(promise, reason) {
+  %promise_internal_reject(promise, reason, true);
+}
+
+%InstallToContext([
+    "promise_reject", DoRejectPromise

[adamk, 2017/01/13 20:31:33]

It looks to me like this is only used by api.cc, so instead of installing this
in the context you could call into the internal one directly there.

[gsathya, 2017/01/13 21:12:40]

Done.

+]);
+
+%OptimizeObjectForAddingMultipleProperties(extrasUtils, 7);
 
 extrasUtils.logStackTrace = function logStackTrace() {
   %DebugTrace();
 };
 
 extrasUtils.log = function log() {
   let message = '';
   for (const arg of arguments) {
     message += arg;
   }
@@ skipping 20 matching lines @@
     return %reflect_apply(func, thisArg, args);
   };
 };
 
 extrasUtils.uncurryThis = function uncurryThis(func) {
   return function(thisArg, ...args) {
     return %reflect_apply(func, thisArg, args);
   };
 };
 
+extrasUtils.rejectPromise = DoRejectPromise;
+
+extrasUtils.markPromiseAsHandled = function(promise) {

[adamk, 2017/01/13 20:31:33]

Make this

function markPromiseAsHandled(...)

to make sure it gets a name property.

[gsathya, 2017/01/13 21:12:40]

Done.

+  %PromiseMarkAsHandled(promise);
+};
+
 %ToFastProperties(extrasUtils);
 
 })