More cleanup related to setting array.length

More cleanup related to setting array.length

BUG=

Committed: https://crrev.com/8076d6ee2de830b6c8be3cefdbdd5d3fa448075e
Cr-Commit-Position: refs/heads/master@{#29152}

[Toon Verwaest, 2015-06-19 11:40:55]

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

[Toon Verwaest, 2015-06-19 11:40:55]

ptal

[Jakob Kummerow, 2015-06-19 13:23:13]

LGTM with a naming nit.

https://codereview.chromium.org/1191313003/diff/120001/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/1191313003/diff/120001/src/objects.cc#newcode...
src/objects.cc:11941: MaybeHandle<Object>
JSArray::ObserveableSetLength(Handle<JSArray> array,
nit: "observable" (one less 'e') is the far more common spelling.

[Toon Verwaest, 2015-06-19 13:25:36]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-06-19 13:25:36]

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/1191313003/#ps220001
(title: "Addressed comment")

[commit-bot: I haz the power, 2015-06-19 13:25:46]

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

[Toon Verwaest, 2015-06-19 13:58:10]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-06-19 13:58:10]

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/1191313003/#ps240001
(title: " ")

[commit-bot: I haz the power, 2015-06-19 13:58:24]

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

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

Committed patchset #13 (id:240001)

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

Patchset 13 (id:??) landed as
https://crrev.com/8076d6ee2de830b6c8be3cefdbdd5d3fa448075e
Cr-Commit-Position: refs/heads/master@{#29152}