Introduce LookupIterator::PropertyOrElement which converts name to index if possible.

Introduce LookupIterator::PropertyOrElement which converts name to index if possible.

BUG=v8:4137
LOG=n

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

[Toon Verwaest, 2015-06-11 13:19:43]

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

[Toon Verwaest, 2015-06-11 13:19:44]

ptal

[Jakob Kummerow, 2015-06-11 15:00:41]

LGTM!

[Toon Verwaest, 2015-06-12 16:03:53]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-06-12 16:03:53]

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

[commit-bot: I haz the power, 2015-06-12 16:04:06]

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

[commit-bot: I haz the power, 2015-06-12 16:30:09]

Committed patchset #5 (id:70001)

[commit-bot: I haz the power, 2015-06-12 16:30:30]

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

[Michael Achenbach, 2015-06-13 07:18:50]

machenbach@chromium.org changed reviewers:
+ hablich@chromium.org, machenbach@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2015-06-13 07:18:51]

This fixes the layout test (which blocked the roll), but now blocks the roll in
the CQ:
https://codereview.chromium.org/1180823004/

[Toon Verwaest, 2015-06-13 09:57:45]

Looking into it.

[Toon Verwaest, 2015-06-13 10:36:06]

The roll was blocked since I added a correctness DCHECK to ensure we never try
to look up indexed properties as named properties. Apparently the *RealNamed*
API methods only have named variants, but were always used by the embedder to
find elements as well. We'd never find them though, since we wouldn't even look
there.

I'm submitting a CL that ensures we also look for elements now:
https://codereview.chromium.org/1177383004/

I guess for all named API functions we should assume they are used similar to
o["name"] where name could also be a number... At least we should make it
uniform between embedder and V8. This matches us up with their expectations for
now...