Use page header information to test for InNewSpace.

src/heap-inl.h

Index: src/heap-inl.h
diff --git a/src/heap-inl.h b/src/heap-inl.h
index bb79b47d480c35c102673d10d8cc0affd85fd983..3bd479d008b9830d367c77c0374bbf0a1ec15512 100644
--- a/src/heap-inl.h
+++ b/src/heap-inl.h
@@ -254,7 +254,7 @@ MaybeObject* Heap::AllocateRawCell() {
 
 
 bool Heap::InNewSpace(Object* object) {
-  bool result = new_space_.Contains(object);
+  bool result = new_space_.PageContains(object);

[Erik Corry, 2011/05/19 07:22:32]

Not mad keen on this name, but I guess it will go away when the transition to
page-based new-space-contains is done.

[Lasse Reichstein, 2011/05/19 08:40:40]

That is the idea. 
If anything, this name is a little too nice. It should stand out like a sore
thumb, kindof like C++ casts.

   ASSERT(!result ||                  // Either not in new space
          gc_state_ != NOT_IN_GC ||   // ... or in the middle of GC
          InToSpace(object));         // ... or in to-space (where we allocate).
@@ -267,6 +267,11 @@ bool Heap::InNewSpace(Address addr) {
 }
 
 
+bool Heap::InNewSpacePage(Address addr) {
+  return new_space_.PageContains(addr);
+}
+
+
 bool Heap::InFromSpace(Object* object) {
   return new_space_.FromSpaceContains(object);
 }