Remove redudant deleted_count argument from EnqueueSpliceRecord

Remove redudant deleted_count argument from EnqueueSpliceRecord

R=rossberg@chromium.org
BUG=

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

[adamk, 2013-06-05 17:22:03]

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

https://codereview.chromium.org/16438010/diff/1/src/objects.cc#newcode10970
src/objects.cc:10970: SetProperty(deleted, isolate->factory()->length_string(),
This can go inside the if (delete_count) block.

I also wonder if the thing you actually want is a static, handlified version of
JSArray::SetElementsLength, which will short-circuit the whole property-setting
machinery.

[rossberg, 2013-06-06 09:12:05]

LGTM with nits

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

https://codereview.chromium.org/16438010/diff/1/src/objects.cc#newcode10963
src/objects.cc:10963: if (delete_count) {
Nit: can we make this del_cnt > 0?

[rafaelw, 2013-06-06 18:44:41]

done

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

https://codereview.chromium.org/16438010/diff/1/src/objects.cc#newcode10963
src/objects.cc:10963: if (delete_count) {
On 2013/06/06 09:12:05, rossberg wrote:
> Nit: can we make this del_cnt > 0?

Done.

https://codereview.chromium.org/16438010/diff/1/src/objects.cc#newcode10970
src/objects.cc:10970: SetProperty(deleted, isolate->factory()->length_string(),
On 2013/06/05 17:22:03, adamk wrote:
> This can go inside the if (delete_count) block.

Done.

> 
> I also wonder if the thing you actually want is a static, handlified version
of
> JSArray::SetElementsLength, which will short-circuit the whole
property-setting
> machinery.

Yeah. Let's wait and see if this shows up as a bottle neck.

[adamk, 2013-06-06 19:15:45]

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