[promises] Remove runtime call from fastpath in PromiseReject

src/js/prologue.js

Index: src/js/prologue.js
diff --git a/src/js/prologue.js b/src/js/prologue.js
index e564eb5cf803f7d1662a4bab272f62471ff8f20b..461691b7bcbcbef166e59b458baf7c45e7948932 100644
--- a/src/js/prologue.js
+++ b/src/js/prologue.js
@@ -266,7 +266,15 @@ utils.PostDebug = PostDebug;
 
 // -----------------------------------------------------------------------
 
-%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();
@@ -307,6 +315,12 @@ extrasUtils.uncurryThis = function uncurryThis(func) {
   };
 };
 
+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);
 
 })