Reland of r20146 "JSObject::NormalizeElements() handlified."

Reland of r20146 "JSObject::NormalizeElements() handlified."

R=verwaest@chromium.org

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

[Igor Sheludko, 2014-03-21 11:10:33]

PTAL

[Toon Verwaest, 2014-03-21 12:01:15]

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

https://codereview.chromium.org/208003002/diff/1/src/objects.cc#newcode4665
src/objects.cc:4665: MUST_USE_RESULT Handle<SeededNumberDictionary>
CopyFastElementsToDictionary(
Remove the MUST_USE_RESULT annotation.

[Toon Verwaest, 2014-03-21 12:01:25]

lgtm otherwise.

[Toon Verwaest, 2014-03-21 12:02:49]

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

https://codereview.chromium.org/208003002/diff/1/src/objects.cc#newcode4682
src/objects.cc:4682: // exceed the capacity of new space, and we would fail
repeatedly
Remove comment and TENURED flag below, that's not needed anymore.

[Igor Sheludko, 2014-03-21 12:05:45]

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

https://codereview.chromium.org/208003002/diff/1/src/objects.cc#newcode4665
src/objects.cc:4665: MUST_USE_RESULT Handle<SeededNumberDictionary>
CopyFastElementsToDictionary(
On 2014/03/21 12:01:15, Toon Verwaest wrote:
> Remove the MUST_USE_RESULT annotation.

Done.

https://codereview.chromium.org/208003002/diff/1/src/objects.cc#newcode4682
src/objects.cc:4682: // exceed the capacity of new space, and we would fail
repeatedly
On 2014/03/21 12:02:49, Toon Verwaest wrote:
> Remove comment and TENURED flag below, that's not needed anymore.

Done.

[Igor Sheludko, 2014-03-21 12:23:46]

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