TransitionElementsKind() and its recursive call chain handlified.

TransitionElementsKind() and its recursive call chain handlified.

R=verwaest@chromium.org

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

[Igor Sheludko, 2014-03-25 14:45:59]

PTAL

[Toon Verwaest, 2014-03-25 15:19:08]

lgtm with comment

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

https://codereview.chromium.org/207553005/diff/1/src/objects.cc#newcode12730
src/objects.cc:12730: // Check if there is potentially a memento behind the
object. If
Should put this in a DisallowHeapAllocation scope

https://codereview.chromium.org/207553005/diff/1/src/objects.cc#newcode12756
src/objects.cc:12756: Handle<AllocationSite> site(memento->GetAllocationSite());
Until here.

[Igor Sheludko, 2014-03-25 15:27:45]

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

https://codereview.chromium.org/207553005/diff/1/src/objects.cc#newcode12730
src/objects.cc:12730: // Check if there is potentially a memento behind the
object. If
On 2014/03/25 15:19:08, Toon Verwaest wrote:
> Should put this in a DisallowHeapAllocation scope

Done.

https://codereview.chromium.org/207553005/diff/1/src/objects.cc#newcode12756
src/objects.cc:12756: Handle<AllocationSite> site(memento->GetAllocationSite());
On 2014/03/25 15:19:08, Toon Verwaest wrote:
> Until here.

Done.

[Igor Sheludko, 2014-03-25 15:33:38]

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