[heap] Fix rescanning live objects when aborting compaction.

src/heap/spaces.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_HEAP_SPACES_H_
 #define V8_HEAP_SPACES_H_
 
 #include "src/allocation.h"
 #include "src/atomic-utils.h"
 #include "src/base/atomicops.h"
@@ skipping 309 matching lines @@
     // This flag is intended to be used for testing. Works only when both
     // FLAG_stress_compaction and FLAG_manual_evacuation_candidates_selection
     // are set. It forces the page to become an evacuation candidate at next
     // candidates selection cycle.
     FORCE_EVACUATION_CANDIDATE_FOR_TESTING,
 
     // The memory chunk is already logically freed, however the actual freeing
     // still has to be performed.
     PRE_FREED,
 
+    // |COMPACTION_WAS_ABORTED|: Indicates that the compaction in this page
+    //   has been aborted to to lack of memory and needs special handling by

[Hannes Payer (out of office), 2015/10/23 13:09:18]

just drop the "to to lack of memory"

[Michael Lippautz, 2015/10/23 13:27:05]

Done.

+    //   the sweeper.
+    COMPACTION_WAS_ABORTED,
+
     // Last flag, keep at bottom.
     NUM_MEMORY_CHUNK_FLAGS
   };
 
   // |kCompactionDone|: Initial compaction state of a |MemoryChunk|.
   // |kCompactingInProgress|:  Parallel compaction is currently in progress.
   // |kCompactingFinalize|: Parallel compaction is done but the chunk needs to
   //   be finalized.
   // |kCompactingAborted|: Parallel compaction has been aborted, which should
   //   for now only happen in OOM scenarios.
@@ skipping 2716 matching lines @@
     count = 0;
   }
   // Must be small, since an iteration is used for lookup.
   static const int kMaxComments = 64;
 };
 #endif
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_HEAP_SPACES_H_