Start adding support for elements to the LookupIterator

Start adding support for elements to the LookupIterator

BUG=v8:4137
LOG=n

Committed: https://crrev.com/16484824b65b6fc4e57c45dd4e055213021fb705
Cr-Commit-Position: refs/heads/master@{#28546}

[Toon Verwaest, 2015-05-19 15:05:48]

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

[Toon Verwaest, 2015-05-19 15:05:48]

PTAL

[Jakob Kummerow, 2015-05-19 17:01:15]

LGTM with a nit.

https://codereview.chromium.org/1144883002/diff/10006/src/elements.cc
File src/elements.cc (right):

https://codereview.chromium.org/1144883002/diff/10006/src/elements.cc#newcode...
src/elements.cc:1531: static uint32_t GetIndexForKeyImpl(FixedArrayBase* store,
uint32_t index) {
nit: "...ForKey..." should take a "uint32_t key" parameter.

[Toon Verwaest, 2015-05-19 17:14:26]

Addressed comments

[Toon Verwaest, 2015-05-20 21:01:23]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-05-20 21:01:24]

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/1144883002/#ps50001
(title: " ")

[commit-bot: I haz the power, 2015-05-20 21:01:42]

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

[commit-bot: I haz the power, 2015-05-20 21:10:31]

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

[commit-bot: I haz the power, 2015-05-20 21:10:32]

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-05-21 11:49:34]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-05-21 11:49:35]

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/1144883002/#ps70001
(title: " ")

[commit-bot: I haz the power, 2015-05-21 11:49:45]

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

[commit-bot: I haz the power, 2015-05-21 11:50:58]

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

[commit-bot: I haz the power, 2015-05-21 11:50:58]

Try jobs failed on following builders:
  v8_linux64_avx2_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel/builds/593)
  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-05-21 11:54:38]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-05-21 11:54: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/1144883002/#ps90001
(title: " ")

[commit-bot: I haz the power, 2015-05-21 11:54:48]

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

[commit-bot: I haz the power, 2015-05-21 12:19:46]

Committed patchset #6 (id:90001)

[commit-bot: I haz the power, 2015-05-21 12:20:04]

Patchset 6 (id:??) landed as
https://crrev.com/16484824b65b6fc4e57c45dd4e055213021fb705
Cr-Commit-Position: refs/heads/master@{#28546}