Remove calls to non-handlified version of GetProperty(name).

Remove calls to non-handlified version of GetProperty(name).

R=ishell@chromium.org

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

[Igor Sheludko, 2014-04-09 09:52:33]

lgtm with nits:

https://codereview.chromium.org/229973004/diff/1/src/runtime.cc
File src/runtime.cc (right):

https://codereview.chromium.org/229973004/diff/1/src/runtime.cc#newcode9352
src/runtime.cc:9352: isolate, value, MakePair(Failure::Exception(), NULL));
Is it OK to return NULL instead of *receiver_handle here?

https://codereview.chromium.org/229973004/diff/1/test/cctest/test-heap.cc
File test/cctest/test-heap.cc (right):

https://codereview.chromium.org/229973004/diff/1/test/cctest/test-heap.cc#new...
test/cctest/test-heap.cc:306:
Object::GetProperty(CcTest::i_isolate()->global_object(), name);
We can use |global| here instead of CcTest::i_isolate()->global_object().

https://codereview.chromium.org/229973004/diff/1/test/cctest/test-heap.cc#new...
test/cctest/test-heap.cc:324:
Object::GetProperty(CcTest::i_isolate()->global_object(), obj_name);
And here.

[Yang, 2014-04-09 10:34:44]

https://codereview.chromium.org/229973004/diff/1/src/runtime.cc
File src/runtime.cc (right):

https://codereview.chromium.org/229973004/diff/1/src/runtime.cc#newcode9352
src/runtime.cc:9352: isolate, value, MakePair(Failure::Exception(), NULL));
On 2014/04/09 09:52:33, Igor Sheludko wrote:
> Is it OK to return NULL instead of *receiver_handle here?

This is fine. We do it elsewhere too.

https://codereview.chromium.org/229973004/diff/1/test/cctest/test-heap.cc
File test/cctest/test-heap.cc (right):

https://codereview.chromium.org/229973004/diff/1/test/cctest/test-heap.cc#new...
test/cctest/test-heap.cc:306:
Object::GetProperty(CcTest::i_isolate()->global_object(), name);
On 2014/04/09 09:52:33, Igor Sheludko wrote:
> We can use |global| here instead of CcTest::i_isolate()->global_object().

Done.

https://codereview.chromium.org/229973004/diff/1/test/cctest/test-heap.cc#new...
test/cctest/test-heap.cc:324:
Object::GetProperty(CcTest::i_isolate()->global_object(), obj_name);
On 2014/04/09 09:52:33, Igor Sheludko wrote:
> And here.

Done.

[Yang, 2014-04-09 12:22:14]

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