Fix LargeObjectSpace::Contains to check if addr is in new space.
authorantonm@chromium.org <antonm@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Thu, 25 Mar 2010 13:10:50 +0000 (13:10 +0000)
committerantonm@chromium.org <antonm@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Thu, 25 Mar 2010 13:10:50 +0000 (13:10 +0000)
commit9477ea8735a2f71682622c509e68db4c47257422
treea8715aa6531c7ff20e38fd45a1a9c295165fe126
parent498b074bd0db2913cf2c9458407c0d340bbcc05e
Fix LargeObjectSpace::Contains to check if addr is in new space.

Otherwise page header check is not quite robust: if there is a smi
at the same offset as Page::is_normal_page field, wrong result would
be returned.

That shouldn't be the problem for paged spaces as objects in those
pages do not span page boundaries and thus cannot mess with ::is_normal_page field.

Review URL: http://codereview.chromium.org/1175001

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@4270 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
src/spaces.cc
test/cctest/test-heap.cc