Handlify JSObject::SetNormalizedProperty methods.

Handlify JSObject::SetNormalizedProperty methods.

R=yangguo@chromium.org

Committed: http://code.google.com/p/v8/source/detail?r=16799

[Yang, 2013-09-17 16:39:45]

lgtm.

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

https://codereview.chromium.org/24096017/diff/1/src/objects.cc#newcode4113
src/objects.cc:4113: RETURN_IF_EMPTY_HANDLE_VALUE(isolate, result,
Handle<Object>());
You could move this check to the TRANSITION case, since we only set the result
there.

Or you could leave it.

[Michael Starzinger, 2013-09-18 16:19:55]

Thanks for the review. Addressed comments. Landing.

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

https://codereview.chromium.org/24096017/diff/1/src/objects.cc#newcode4113
src/objects.cc:4113: RETURN_IF_EMPTY_HANDLE_VALUE(isolate, result,
Handle<Object>());
On 2013/09/17 16:39:46, Yang wrote:
> You could move this check to the TRANSITION case, since we only set the result
> there.
> 
> Or you could leave it.

Done.

[Michael Starzinger, 2013-09-18 16:26:02]

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