Fix spurious bugs in GrowAndShrinkNewSpace when the

test/cctest/test-heap.cc

Index: test/cctest/test-heap.cc
===================================================================
--- test/cctest/test-heap.cc	(revision 11455)
+++ test/cctest/test-heap.cc	(working copy)
@@ -1275,6 +1275,12 @@
   InitializeVM();
   NewSpace* new_space = HEAP->new_space();
 
+  if (HEAP->ReservedSemiSpaceSize() == HEAP->InitialSemiSpaceSize()) {

[Michael Starzinger, 2012/04/26 14:41:45]

Shouldn't the test say something like "InitialSemiSpaceSize ==
MaxSemiSpaceSize"? Because the current condition looks more like a tautology.

+    // We can't test new space growing and shrinking if the max size
+    // is the same as the minimum (initial) size.
+    return;
+  }
+
   // Explicitly growing should double the space capacity.
   intptr_t old_capacity, new_capacity;
   old_capacity = new_space->Capacity();
@@ -1315,6 +1321,13 @@
 
 TEST(CollectingAllAvailableGarbageShrinksNewSpace) {
   InitializeVM();
+
+  if (HEAP->ReservedSemiSpaceSize() == HEAP->InitialSemiSpaceSize()) {
+    // We can't test new space growing and shrinking if the max size
+    // is the same as the minimum (initial) size.
+    return;
+  }
+
   v8::HandleScope scope;
   NewSpace* new_space = HEAP->new_space();
   intptr_t old_capacity, new_capacity;