Use RETURN_ON_EXCEPTION rather than .Assert when setting .length, which may fail due to stack overf…

Use RETURN_ON_EXCEPTION rather than .Assert when setting .length, which may fail due to stack overflow.

BUG=449447
LOG=n

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

[Toon Verwaest, 2015-01-20 13:34:01]

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

[Toon Verwaest, 2015-01-20 13:34:02]

PTAL

[Yang, 2015-01-20 13:37:46]

On 2015/01/20 13:34:02, Toon Verwaest wrote:
> PTAL

driveby: how about adding a test case?

[Toon Verwaest, 2015-01-20 13:57:26]

The existing test-case is an infinite loop if it doesn't crash. It's tricky to
construct one that doesn't do that.

[Yang, 2015-01-20 14:13:26]

On 2015/01/20 13:57:26, Toon Verwaest wrote:
> The existing test-case is an infinite loop if it doesn't crash. It's tricky to
> construct one that doesn't do that.

nvm. lgtm.

[Toon Verwaest, 2015-01-20 14:15:16]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2015-01-20 14:16:01]

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

[commit-bot: I haz the power, 2015-01-20 15:02:42]

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

[commit-bot: I haz the power, 2015-01-20 15:02:43]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8
(http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/80)

[Toon Verwaest, 2015-01-20 15:25:50]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2015-01-20 15:26:20]

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

[commit-bot: I haz the power, 2015-01-20 16:08:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-01-20 16:08:33]

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