Adds the possibility of setting a Code object as the callback of a FunctionTemplate.

src/api-natives.cc

Index: src/api-natives.cc
diff --git a/src/api-natives.cc b/src/api-natives.cc
index d8dd1510417641197da707cf27f79e51586886d3..b14bfbb4c71b52fbb0a001ae2f4228282dc68fa1 100644
--- a/src/api-natives.cc
+++ b/src/api-natives.cc
@@ -438,7 +438,15 @@ void ApiNatives::AddNativeDataProperty(Isolate* isolate,
 Handle<JSFunction> ApiNatives::CreateApiFunction(
     Isolate* isolate, Handle<FunctionTemplateInfo> obj,
     Handle<Object> prototype, ApiInstanceType instance_type) {
-  Handle<Code> code = isolate->builtins()->HandleApiCall();
+  Handle<Code> code;
+  if (obj->call_code()->IsCallHandlerInfo() &&
+      !handle(CallHandlerInfo::cast(obj->call_code()))

[Toon Verwaest, 2015/11/18 08:51:46]

Why is this wrapped in a handle before calling fast_handler?

[epertoso, 2015/11/20 15:27:13]

Removed.

+           ->fast_handler()
+           ->IsUndefined()) {

[vogelheim, 2015/11/18 17:30:05]

nitpick: Why !..->IsUndefined(), instead of ...->IsCode() ?

(They do the same thing, it's just with the negation being 2 lines above the
thing to be negated I at first just overlooked it.)

[epertoso, 2015/11/20 15:27:13]

Done.

+    code = isolate->builtins()->HandleFastApiCall();
+  } else {
+    code = isolate->builtins()->HandleApiCall();
+  }
   Handle<Code> construct_stub = isolate->builtins()->JSConstructStubApi();
 
   obj->set_instantiated(true);