Allow JavaScript accessors on API objects.

src/api-natives.h

 // 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.
 
 #ifndef V8_API_NATIVES_H_
 #define V8_API_NATIVES_H_
 
 #include "src/handles.h"
 #include "src/property-details.h"
 
@@ skipping 32 matching lines @@
   static void AddDataProperty(Isolate* isolate, Handle<TemplateInfo> info,
                               Handle<Name> name, Handle<Object> value,
                               PropertyAttributes attributes);
 
   static void AddAccessorProperty(Isolate* isolate, Handle<TemplateInfo> info,
                                   Handle<Name> name,
                                   Handle<FunctionTemplateInfo> getter,
                                   Handle<FunctionTemplateInfo> setter,
                                   PropertyAttributes attributes);
 
+#ifdef V8_JS_ACCESSORS
+  static void AddAccessorProperty(Isolate* isolate, Handle<TemplateInfo> info,

[epertoso, 2015/09/24 12:09:06]

These two overloads of AddAccessorProperty are internal only. Since down the
path we throw away the type-safety anyway, and their code is essentially the
same, how about just having one variant with type Handle<Object> for the getter
and setter arguments?

[vogelheim, 2015/09/24 16:35:49]

Done.

+                                  Handle<Name> name, Handle<JSFunction> getter,
+                                  Handle<JSFunction> setter,
+                                  PropertyAttributes attributes);
+#endif  // V8_JS_ACCESSORS
+
   static void AddNativeDataProperty(Isolate* isolate, Handle<TemplateInfo> info,
                                     Handle<AccessorInfo> property);
 };
 
 }  // namespace internal
 }  // namespace v8
 
 #endif