Fix slot buffers of abandoned evacuation candidates.

src/mark-compact.cc

Index: src/mark-compact.cc
diff --git a/src/mark-compact.cc b/src/mark-compact.cc
index 956d0c953a0dbddcb3adcae3eadc80037cf10297..0196349511c983330c96942e160fd8f3bb6c6556 100644
--- a/src/mark-compact.cc
+++ b/src/mark-compact.cc
@@ -2690,8 +2690,10 @@ void MarkCompactCollector::EvacuatePages() {
         // Without room for expansion evacuation is not guaranteed to succeed.
         // Pessimistically abandon unevacuated pages.
         for (int j = i; j < npages; j++) {
-          evacuation_candidates_[j]->ClearEvacuationCandidate();
-          evacuation_candidates_[j]->SetFlag(Page::RESCAN_ON_EVACUATION);
+          p = evacuation_candidates_[j];

[Vyacheslav Egorov (Chromium), 2011/09/27 13:24:56]

it's a bit shady to reuse p from the outer scope here, might lead to confusion
in future.

consider having a separate variable.

[Michael Starzinger, 2011/09/27 13:34:43]

Done.

+          slots_buffer_allocator_.DeallocateChain(p->slots_buffer_address());
+          p->ClearEvacuationCandidate();
+          p->SetFlag(Page::RESCAN_ON_EVACUATION);
         }
         return;
       }