Fix pointer iteration for maps.

Fix pointer iteration for maps.

BUG=
R=hpayer@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=23114

[Jarin, 2014-08-13 19:26:15]

This should avoid treating instance_size field in a map as a heap pointer (this
was introduced by r22208).

[Hannes Payer (out of office), 2014-08-13 19:52:52]

Awesome, thanks for fixing. The patch looks good, but could you also add a
regression test?

[Jarin, 2014-08-14 07:18:33]

On 2014/08/13 19:52:52, Hannes Payer wrote:
> Awesome, thanks for fixing. The patch looks good, but could you also add a
> regression test?

I have added a test (and checked it failed before the fix). Does it make sense?

[Hannes Payer (out of office), 2014-08-14 07:27:32]

lgtm, with one comment nit

https://codereview.chromium.org/475463003/diff/20001/test/cctest/test-heap.cc
File test/cctest/test-heap.cc (right):

https://codereview.chromium.org/475463003/diff/20001/test/cctest/test-heap.cc...
test/cctest/test-heap.cc:4501: // the map.
Can you just add a bit more of information why we do that?

[Hannes Payer (out of office), 2014-08-14 07:27:32]

lgtm, with one comment nit

https://codereview.chromium.org/475463003/diff/20001/test/cctest/test-heap.cc
File test/cctest/test-heap.cc (right):

https://codereview.chromium.org/475463003/diff/20001/test/cctest/test-heap.cc...
test/cctest/test-heap.cc:4501: // the map.
Can you just add a bit more of information why we do that?

[Jarin, 2014-08-14 07:37:33]

https://codereview.chromium.org/475463003/diff/20001/test/cctest/test-heap.cc
File test/cctest/test-heap.cc (right):

https://codereview.chromium.org/475463003/diff/20001/test/cctest/test-heap.cc...
test/cctest/test-heap.cc:4501: // the map.
On 2014/08/14 07:27:32, Hannes Payer wrote:
> Can you just add a bit more of information why we do that?

Done. Better?

[Jarin, 2014-08-14 07:41:40]

Committed patchset #3 manually as 23114 (presubmit successful).