Split setting array length from handling new Array(non-number)

Split setting array length from handling new Array(non-number)

BUG=

Committed: https://crrev.com/be8528b3bda429bcd2670b8ef39e37d9c341ef13
Cr-Commit-Position: refs/heads/master@{#29144}

[Toon Verwaest, 2015-06-18 14:05:17]

verwaest@chromium.org changed reviewers:
+ jkummerow@chromium.org

[Toon Verwaest, 2015-06-18 14:05:18]

ptal

There's more cleanup to be done, but lets start with this.

[Jakob Kummerow, 2015-06-18 19:22:43]

LGTM!

https://codereview.chromium.org/1193673002/diff/1/src/elements.cc
File src/elements.cc (right):

https://codereview.chromium.org/1193673002/diff/1/src/elements.cc#newcode1728
src/elements.cc:1728: // even though the proposed length was a smi, new_length
could
nit: while you're here, s/even/Even/

[Toon Verwaest, 2015-06-19 09:08:39]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-06-19 09:08:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/1193673002/#ps20001
(title: "Addressed nit")

[commit-bot: I haz the power, 2015-06-19 09:08:55]

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

[commit-bot: I haz the power, 2015-06-19 09:18:29]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-19 09:18:30]

Try jobs failed on following builders:
  v8_linux_gcc_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)

[Toon Verwaest, 2015-06-19 10:57:06]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-06-19 10:57:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/1193673002/#ps40001
(title: "Make GCC happy")

[commit-bot: I haz the power, 2015-06-19 10:57:19]

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

[commit-bot: I haz the power, 2015-06-19 11:21:32]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-06-19 11:21:48]

Patchset 3 (id:??) landed as
https://crrev.com/be8528b3bda429bcd2670b8ef39e37d9c341ef13
Cr-Commit-Position: refs/heads/master@{#29144}