Handlify GetPropertyWithFailedAccessCheck.

src/objects.h

Index: src/objects.h
diff --git a/src/objects.h b/src/objects.h
index 958201b74c4e48b32c5c66092bf6ad0f3a622db5..81af3a48eef7c1a46a26efeb8095e016662404ab 100644
--- a/src/objects.h
+++ b/src/objects.h
@@ -2117,6 +2117,11 @@ class JSObject: public JSReceiver {
   // a dictionary, and it will stay a dictionary.
   MUST_USE_RESULT MaybeObject* PrepareSlowElementsForSort(uint32_t limit);
 
+  static Handle<Object> GetPropertyWithCallback(Handle<JSObject> holder,

[Michael Starzinger, 2013/10/02 18:39:53]

nit: s/holder/object/ for consistency.

+                                                Handle<Object> receiver,
+                                                Handle<Object> structure,
+                                                Handle<Name> name);
+
   MUST_USE_RESULT MaybeObject* GetPropertyWithCallback(Object* receiver,
                                                        Object* structure,
                                                        Name* name);
@@ -2243,10 +2248,11 @@ class JSObject: public JSReceiver {
                                     Handle<AccessorInfo> info);
 
   // Used from Object::GetProperty().
-  MUST_USE_RESULT MaybeObject* GetPropertyWithFailedAccessCheck(
-      Object* receiver,
+  static Handle<Object> GetPropertyWithFailedAccessCheck(
+      Handle<JSObject> failed_check_obj,

[Michael Starzinger, 2013/10/02 18:39:53]

nit: s/failed_check_obj/object/ for consistency.

Also I think this should be made private and the Object class should be made a
friend of JSObject (similar to what we did with JSReceiver). This will prevent
this entry point from being abused from outside.

+      Handle<Object> receiver,
       LookupResult* result,
-      Name* name,
+      Handle<Name> name,
       PropertyAttributes* attributes);
   static Handle<Object> GetPropertyWithInterceptor(
       Handle<JSObject> holder,