Handlify GetPropertyWithFailedAccessCheck.

src/objects.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 2099 matching lines @@
   MUST_USE_RESULT inline MaybeObject* EnsureWritableFastElements();
 
   // Collects elements starting at index 0.
   // Undefined values are placed after non-undefined values.
   // Returns the number of non-undefined values.
   MUST_USE_RESULT MaybeObject* PrepareElementsForSort(uint32_t limit);
   // As PrepareElementsForSort, but only on objects where elements is
   // 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);
 
   static Handle<Object> SetPropertyWithCallback(
       Handle<JSObject> object,
       Handle<Object> structure,
       Handle<Name> name,
       Handle<Object> value,
       Handle<JSObject> holder,
@@ skipping 106 matching lines @@
                              Handle<Object> getter,
                              Handle<Object> setter,
                              PropertyAttributes attributes,
                              v8::AccessControl access_control = v8::DEFAULT);
 
   // Defines an AccessorInfo property on the given object.
   static Handle<Object> SetAccessor(Handle<JSObject> object,
                                     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,
       Handle<Object> receiver,
       Handle<Name> name,
       PropertyAttributes* attributes);
   static Handle<Object> GetPropertyPostInterceptor(
       Handle<JSObject> holder,
       Handle<Object> receiver,
       Handle<Name> name,
@@ skipping 7982 matching lines @@
     } else {
       value &= ~(1 << bit_position);
     }
     return value;
   }
 };
 
 } }  // namespace v8::internal
 
 #endif  // V8_OBJECTS_H_