Reland of "[cctest] Add tests for aborting compaction of pages"

src/heap/mark-compact.cc

Index: src/heap/mark-compact.cc
diff --git a/src/heap/mark-compact.cc b/src/heap/mark-compact.cc
index 4f16a3ae3815c0356bf83d22578795dc6f919aa7..70c38f8ebb96bfa6dfdec6e857cd3e10cacc971a 100644
--- a/src/heap/mark-compact.cc
+++ b/src/heap/mark-compact.cc
@@ -3774,6 +3774,14 @@ void MarkCompactCollector::StartSweepSpace(PagedSpace* space) {
       continue;
     }
 
+    if (p->IsFlagSet(Page::NEVER_ALLOCATE_ON_PAGE)) {
+      // We do not sweep pages that are marked to be ignored for allocation.
+      p->SetWasSwept();
+      Bitmap::Clear(p);
+      p->ResetLiveBytes();

[Hannes Payer (out of office), 2015/12/16 12:07:14]

Should we really reset live bytes here? That would indicate that everything is
dead on this page.

[Michael Lippautz, 2015/12/16 12:13:14]

We reset live bytes after sweeping a page. I think we want to do that here since
we don't sweep them but want them to be in the same state as regular pages after
sweeping (i.e. cleared mark bits, reset live byte counter).

[Hannes Payer (out of office), 2015/12/16 12:44:00]

Acknowledged.

+      continue;
+    }
+
     // One unused page is kept, all further are released before sweeping them.
     if (p->LiveBytes() == 0) {
       if (unused_page_present) {