[api] Expose FunctionCallbackInfo::NewTarget

src/builtins.h

Index: src/builtins.h
diff --git a/src/builtins.h b/src/builtins.h
index e01f95916db794b8251576bf6ff8183f93f808bc..94277cb8aad26ee9ca898f26f191ae79e34dd972 100644
--- a/src/builtins.h
+++ b/src/builtins.h
@@ -161,7 +161,7 @@ inline bool operator&(BuiltinExtraArguments lhs, BuiltinExtraArguments rhs) {
   V(SymbolConstructor_ConstructStub, kTarget)                  \
                                                                \
   V(HandleApiCall, kTarget)                                    \
-  V(HandleApiCallConstruct, kTarget)                           \
+  V(HandleApiCallConstruct, kTargetAndNewTarget)               \

[Benedikt Meurer, 2016/04/23 11:57:47]

This is not sufficient, as the callsite is custom code in builtins-x64.cc and so
you need to make sure to manually add NewTarget there.

[Benedikt Meurer, 2016/04/23 18:21:45]

Actually this is nonsense. The JSConstructStubHelper passes NewTarget in the
appropriate register and the builtin adaptor pushes it, so everything works fine
the way it is currently.

[adamk, 2016/04/25 22:59:55]

Yup, this seems to work just fine.

   V(HandleApiCallAsFunction, kNone)                            \
   V(HandleApiCallAsConstructor, kNone)                         \
                                                                \