Map v8::Object to v8::internal::JSReceiver

src/api.h

Index: src/api.h
diff --git a/src/api.h b/src/api.h
index 08fbd7ee8f389d1abc0ae788535aa6ffa4888bca..6f8c58043b5b93e37511c17f29f001e7464e2c21 100644
--- a/src/api.h
+++ b/src/api.h
@@ -145,7 +145,7 @@ class RegisteredExtension {
   V(TypeSwitch, TypeSwitchInfo)              \
   V(Data, Object)                            \
   V(RegExp, JSRegExp)                        \
-  V(Object, JSObject)                        \
+  V(Object, JSReceiver)                      \
   V(Array, JSArray)                          \
   V(Map, JSMap)                              \
   V(Set, JSSet)                              \
@@ -200,8 +200,6 @@ class Utils {
       v8::internal::Handle<v8::internal::Symbol> obj);
   static inline Local<RegExp> ToLocal(
       v8::internal::Handle<v8::internal::JSRegExp> obj);
-  static inline Local<Object> ToLocal(
-      v8::internal::Handle<v8::internal::JSObject> obj);
   static inline Local<Array> ToLocal(
       v8::internal::Handle<v8::internal::JSArray> obj);
   static inline Local<Map> ToLocal(
@@ -239,6 +237,8 @@ class Utils {
   static inline Local<SharedArrayBuffer> ToLocalShared(
       v8::internal::Handle<v8::internal::JSArrayBuffer> obj);
 
+  static inline Local<Object> ReceiverToLocal(

[rossberg, 2015/11/04 10:43:56]

Is there a reason to name this specially? So far, we only do that for functions
whose argument type does not uniquely/canonically determine the result type.

[rossberg, 2015/11/09 13:31:46]

What about this one?

[jochen (gone - plz use gerrit), 2015/11/09 22:42:04]

that's the case here - JSReceiver could also be a v8::Function()

[rossberg, 2015/11/10 07:04:48]

I don't understand. Not all receivers are functions, so that mapping would not
make sense as a generic choice. v8::Object is the most specific API type that
can represent all receivers.

+      v8::internal::Handle<v8::internal::JSReceiver> obj);
   static inline Local<Message> MessageToLocal(
       v8::internal::Handle<v8::internal::Object> obj);
   static inline Local<Promise> PromiseToLocal(
@@ -353,7 +353,6 @@ MAKE_TO_LOCAL(ToLocal, Name, Name)
 MAKE_TO_LOCAL(ToLocal, String, String)
 MAKE_TO_LOCAL(ToLocal, Symbol, Symbol)
 MAKE_TO_LOCAL(ToLocal, JSRegExp, RegExp)
-MAKE_TO_LOCAL(ToLocal, JSObject, Object)
 MAKE_TO_LOCAL(ToLocal, JSArray, Array)
 MAKE_TO_LOCAL(ToLocal, JSMap, Map)
 MAKE_TO_LOCAL(ToLocal, JSSet, Set)
@@ -370,6 +369,7 @@ MAKE_TO_LOCAL(ToLocal, ObjectTemplateInfo, ObjectTemplate)
 MAKE_TO_LOCAL(SignatureToLocal, FunctionTemplateInfo, Signature)
 MAKE_TO_LOCAL(AccessorSignatureToLocal, FunctionTemplateInfo, AccessorSignature)
 MAKE_TO_LOCAL(ToLocal, TypeSwitchInfo, TypeSwitch)
+MAKE_TO_LOCAL(ReceiverToLocal, JSReceiver, Object)

[rossberg, 2015/11/04 10:43:56]

Same here.

 MAKE_TO_LOCAL(MessageToLocal, Object, Message)
 MAKE_TO_LOCAL(PromiseToLocal, JSObject, Promise)
 MAKE_TO_LOCAL(StackTraceToLocal, JSArray, StackTrace)