Handlify GetPropertyWithFailedAccessCheck.

Handlify GetPropertyWithFailedAccessCheck.

R=mstarzinger@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=17146

[Yang, 2013-10-02 14:35:24]

Please take a look.

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

LGTM if comments are addressed.

https://codereview.chromium.org/25732002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/25732002/diff/1/src/objects.cc#newcode510
src/objects.cc:510: Handle<Object>
JSObject::GetPropertyWithCallback(Handle<JSObject> holder,
nit: s/holder/object/ for consistency.

https://codereview.chromium.org/25732002/diff/1/src/objects.cc#newcode524
src/objects.cc:524: Handle<JSObject> failed_check_obj,
nit: s/failed_check_obj/object/ for consistency.

https://codereview.chromium.org/25732002/diff/1/src/objects.h
File src/objects.h (right):

https://codereview.chromium.org/25732002/diff/1/src/objects.h#newcode2120
src/objects.h:2120: static Handle<Object>
GetPropertyWithCallback(Handle<JSObject> holder,
nit: s/holder/object/ for consistency.

https://codereview.chromium.org/25732002/diff/1/src/objects.h#newcode2252
src/objects.h:2252: Handle<JSObject> failed_check_obj,
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.

[Yang, 2013-10-10 15:39:01]

Committed patchset #2 manually as r17146 (presubmit successful).