Implement LookupIterator designed to replace LookupResult

src/objects-inl.h

Index: src/objects-inl.h
diff --git a/src/objects-inl.h b/src/objects-inl.h
index cd61ab52b92a657d66eaef1d2b759e3123584d0d..2bca797d18a8bc5d33c839e842a0474a33530dbc 100644
--- a/src/objects-inl.h
+++ b/src/objects-inl.h
@@ -29,6 +29,7 @@
 #include "src/incremental-marking.h"
 #include "src/transitions-inl.h"
 #include "src/objects-visiting.h"
+#include "src/lookup.h"
 
 namespace v8 {
 namespace internal {
@@ -664,8 +665,7 @@ bool Object::IsJSObject() {
 
 bool Object::IsJSProxy() {
   if (!Object::IsHeapObject()) return false;
-  InstanceType type = HeapObject::cast(this)->map()->instance_type();
-  return FIRST_JS_PROXY_TYPE <= type && type <= LAST_JS_PROXY_TYPE;
+  return  HeapObject::cast(this)->map()->IsJSProxyMap();

[rossberg, 2014/06/11 15:37:00]

Why not make the same change to IsJSObject? (And perhaps IsJSReceiver.)

[Toon Verwaest, 2014/06/11 17:32:18]

I'll defer that for now, not really relevant for this CL.

On 2014/06/11 15:37:00, rossberg wrote:

 }
 
 
@@ -1053,8 +1053,8 @@ bool Object::HasSpecificClassOf(String* name) {
 
 MaybeHandle<Object> Object::GetProperty(Handle<Object> object,
                                         Handle<Name> name) {
-  PropertyAttributes attributes;
-  return GetPropertyWithReceiver(object, object, name, &attributes);
+  LookupIterator it(object, name);
+  return GetProperty(&it);
 }