[Binding] Get non indexed property names for dense array serialization

Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the performance.)

This CL improves a micro-benchmark score of PerformanceTests/Bindings/serialize-array.html
from 430ms to 260ms on my local machine.
https://chromeperf.appspot.com/report?sid=96d5e4ae4e5a4085541ab88e8672b8836d57d223b759a3d5406a06c0b1a65420

BUG=148757
Committed: https://crrev.com/212bfd8b6f448dcb84a4bb2f46d26c7e1cb28f64
Cr-Commit-Position: refs/heads/master@{#398001}

[peria, 2016-06-02 09:36:54]

Description was changed from

==========
Re-get non indexed property names for dense array serialization

BUG=148757
==========

to

==========
Get non indexed property names again for dense array serialization.

This change improves a microbenchmark
https://1ede9c8b4261401ce30e527057727688b46ce0d9.googledrive.com/host/0B4QUVw...
from 340ms to 230ms.


* if we assume all arrays are DenseArray, we can skip to call
GetOwnPropertyNames(), and it improves the micro benchmark more, to 115ms.


NOCOMMIT=true
BUG=148757
==========

[peria, 2016-06-06 08:32:08]

Description was changed from

==========
Get non indexed property names again for dense array serialization.

This change improves a microbenchmark
https://1ede9c8b4261401ce30e527057727688b46ce0d9.googledrive.com/host/0B4QUVw...
from 340ms to 230ms.


* if we assume all arrays are DenseArray, we can skip to call
GetOwnPropertyNames(), and it improves the micro benchmark more, to 115ms.


NOCOMMIT=true
BUG=148757
==========

to

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score
https://1ede9c8b4261401ce30e527057727688b46ce0d9.googledrive.com/host/0B4QUVw...
from 340ms to 230ms.


BUG=148757
==========

[peria, 2016-06-06 08:35:06]

peria@chromium.org changed reviewers:
+ haraken@chromium.org, yukishiino@chromium.org

[peria, 2016-06-06 08:35:07]

PTL

Thanks to V8 team's work, we can get only non indexed property names of arrays.

[haraken, 2016-06-06 08:38:12]

LGTM, but do you have a benchmark (in PerformanceTests/Bindings/) to measure
performance impacts of your work?

[peria, 2016-06-06 08:44:32]

Description was changed from

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score
https://1ede9c8b4261401ce30e527057727688b46ce0d9.googledrive.com/host/0B4QUVw...
from 340ms to 230ms.


BUG=148757
==========

to

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score of
PerformanceTests/Bindings/serialize-array.html
from 430ms to 260ms on my local machine.


BUG=148757
==========

[peria, 2016-06-06 08:45:29]

On 2016/06/06 08:38:12, haraken wrote:
> LGTM, but do you have a benchmark (in PerformanceTests/Bindings/) to measure
> performance impacts of your work?

Yes, PerformanceTests/Bindings/serialize-array.html is it.
You can track it with
https://chromeperf.appspot.com/report?sid=96d5e4ae4e5a4085541ab88e8672b8836d5...
I updated the description.

[peria, 2016-06-06 08:45:50]

Description was changed from

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score of
PerformanceTests/Bindings/serialize-array.html
from 430ms to 260ms on my local machine.


BUG=148757
==========

to

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score of
PerformanceTests/Bindings/serialize-array.html
from 430ms to 260ms on my local machine.
https://chromeperf.appspot.com/report?sid=96d5e4ae4e5a4085541ab88e8672b8836d5...

BUG=148757
==========

[Yuki, 2016-06-06 08:48:33]

lgtm

[peria, 2016-06-06 08:49:12]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-06-06 08:49:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2037583002/20001

[commit-bot: I haz the power, 2016-06-06 10:26:41]

Description was changed from

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score of
PerformanceTests/Bindings/serialize-array.html
from 430ms to 260ms on my local machine.
https://chromeperf.appspot.com/report?sid=96d5e4ae4e5a4085541ab88e8672b8836d5...

BUG=148757
==========

to

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score of
PerformanceTests/Bindings/serialize-array.html
from 430ms to 260ms on my local machine.
https://chromeperf.appspot.com/report?sid=96d5e4ae4e5a4085541ab88e8672b8836d5...

BUG=148757
==========

[commit-bot: I haz the power, 2016-06-06 10:26:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-06 10:27:57]

Description was changed from

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score of
PerformanceTests/Bindings/serialize-array.html
from 430ms to 260ms on my local machine.
https://chromeperf.appspot.com/report?sid=96d5e4ae4e5a4085541ab88e8672b8836d5...

BUG=148757
==========

to

==========
Get non indexed property names for dense array serialization.

Although we already have all own property names to judge if the serializing
array is dense or sparse, getting non indexed ones again for dense arrays
improves performance of serialization of large scale dense arrays.
(For small size arrays and sparse arrays, this CL does not change the
performance.)

This CL improves a micro-benchmark score of
PerformanceTests/Bindings/serialize-array.html
from 430ms to 260ms on my local machine.
https://chromeperf.appspot.com/report?sid=96d5e4ae4e5a4085541ab88e8672b8836d5...

BUG=148757

Committed: https://crrev.com/212bfd8b6f448dcb84a4bb2f46d26c7e1cb28f64
Cr-Commit-Position: refs/heads/master@{#398001}
==========

[commit-bot: I haz the power, 2016-06-06 10:27:58]

Patchset 2 (id:??) landed as
https://crrev.com/212bfd8b6f448dcb84a4bb2f46d26c7e1cb28f64
Cr-Commit-Position: refs/heads/master@{#398001}