[LookupIterator] Add fast-path for reloading accessor property information

[LookupIterator] Add fast-path for reloading accessor property information

This is the equivalent of the fast-path in
ApplyTransitionToDataProperty.

BUG=588893, 325923
LOG=n

Committed: https://crrev.com/7011ddd6528ce75491228efc001790090a8bb947
Cr-Commit-Position: refs/heads/master@{#34488}

[Toon Verwaest, 2016-03-04 09:55:10]

verwaest@chromium.org changed reviewers:
+ cbruni@chromium.org, eisinger@chromium.org, peria@chromium.org

[Toon Verwaest, 2016-03-04 09:55:10]

cbruni: ptal
eisinger, peria: fyi

[jochen (gone - plz use gerrit), 2016-03-04 09:58:02]

jochen@chromium.org changed reviewers:
+ jochen@chromium.org
- eisinger@chromium.org

[jochen (gone - plz use gerrit), 2016-03-04 09:58:03]

https://codereview.chromium.org/1765633003/diff/1/src/lookup.cc
File src/lookup.cc (right):

https://codereview.chromium.org/1765633003/diff/1/src/lookup.cc#newcode375
src/lookup.cc:375: int number = new_map->LastAdded();
why's LastAdded not return uint32_t?

[Toon Verwaest, 2016-03-04 10:00:24]

Traditionally descriptor lookup has always been int. I never updated it since
the fast that any descriptor lookup (Search) uses -1 as kNotFound; and the
DescriptorLookupCache uses -2; but the LookupIterator uses a different mechanism
so there we just use uint32_t. DescriptorArrays always have a smi-length, so
we're int-bound anyway (max 2^10-2 descriptors iirc).

[jochen (gone - plz use gerrit), 2016-03-04 10:01:27]

lgtm

[Toon Verwaest, 2016-03-04 10:02:10]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2016-03-04 10:02:16]

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

[commit-bot: I haz the power, 2016-03-04 10:39:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-04 10:41:24]

Description was changed from

==========
[LookupIterator] Add fast-path for reloading accessor property information

This is the equivalent of the fast-path in
ApplyTransitionToDataProperty.

BUG=588893, 325923
LOG=n
==========

to

==========
[LookupIterator] Add fast-path for reloading accessor property information

This is the equivalent of the fast-path in
ApplyTransitionToDataProperty.

BUG=588893, 325923
LOG=n

Committed: https://crrev.com/7011ddd6528ce75491228efc001790090a8bb947
Cr-Commit-Position: refs/heads/master@{#34488}
==========

[commit-bot: I haz the power, 2016-03-04 10:41:25]

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