[builtins] Take fast path in Array.prototype.keys() only if length is an Smi

[builtins] Take fast path in Array.prototype.keys() only if length is an Smi

Do not take the fast path for Array.prototype.keys() if the array length is not
guaranteed to be an Smi.

BUG=660925, v8:5388
R=bmeurer@chromium.org, mstarzinger@chromium.org

Committed: https://crrev.com/2a350ed4573252328c9e150f1c8ff8874703254a
Cr-Commit-Position: refs/heads/master@{#40976}

[caitp, 2016-11-14 18:00:43]

The CQ bit was checked by caitp@igalia.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-14 18:00:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-14 18:26:57]

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

[commit-bot: I haz the power, 2016-11-14 18:26:57]

Dry run: This issue passed the CQ dry run.

[caitp, 2016-11-14 18:46:41]

Take a look when you get some time.

[Benedikt Meurer, 2016-11-14 18:49:21]

Nice, LGTM. Thanks.

[caitp, 2016-11-14 18:49:52]

The CQ bit was checked by caitp@igalia.com

[commit-bot: I haz the power, 2016-11-14 18:50:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-14 18:52:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 22:33:13]

Description was changed from

==========
[builtins] Take fast path in Array.prototype.keys() only if length is an Smi

Do not take the fast path for Array.prototype.keys() if the array length is not
guaranteed to be an Smi.

BUG=660925, v8:5388
R=bmeurer@chromium.org, mstarzinger@chromium.org
==========

to

==========
[builtins] Take fast path in Array.prototype.keys() only if length is an Smi

Do not take the fast path for Array.prototype.keys() if the array length is not
guaranteed to be an Smi.

BUG=660925, v8:5388
R=bmeurer@chromium.org, mstarzinger@chromium.org

Committed: https://crrev.com/2a350ed4573252328c9e150f1c8ff8874703254a
Cr-Commit-Position: refs/heads/master@{#40976}
==========

[commit-bot: I haz the power, 2016-11-17 22:33:14]

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