[heap] Abort AdvanceIncrementalMarking when we do not make marking progress.

src/heap/incremental-marking.cc

Index: src/heap/incremental-marking.cc
diff --git a/src/heap/incremental-marking.cc b/src/heap/incremental-marking.cc
index e49b1f1ec304640041e5a4dfa74042f9c59087f2..5194e8b24ec154f58394f164566e978a3e5df52a 100644
--- a/src/heap/incremental-marking.cc
+++ b/src/heap/incremental-marking.cc
@@ -1030,12 +1030,15 @@ double IncrementalMarking::AdvanceIncrementalMarking(
   }
 
   double remaining_time_in_ms = 0.0;
+  intptr_t bytes_processed = 0;
   do {
-    Step(step_size_in_bytes, step_actions.completion_action,
-         step_actions.force_marking, step_actions.force_completion);
+    bytes_processed =
+        Step(step_size_in_bytes, step_actions.completion_action,
+             step_actions.force_marking, step_actions.force_completion);
     remaining_time_in_ms =
         deadline_in_ms - heap()->MonotonicallyIncreasingTimeInMs();
-  } while (remaining_time_in_ms >=
+  } while (bytes_processed > 0 &&
+           remaining_time_in_ms >=
                2.0 * GCIdleTimeHandler::kIncrementalMarkingStepTimeInMs &&
            !IsComplete() &&
            !heap()->mark_compact_collector()->marking_deque()->IsEmpty());
@@ -1183,7 +1186,8 @@ intptr_t IncrementalMarking::Step(intptr_t allocated_bytes,
         bytes_scanned_ = 0;
         StartMarking();
       }
-    } else if (state_ == MARKING) {

[Hannes Payer (out of office), 2016/04/01 12:48:24]

This is a slight policy change. However, I don't think there is much harm when
we do steps after sweeping.

+    }
+    if (state_ == MARKING) {
       bytes_processed = ProcessMarkingDeque(bytes_to_process);
       if (heap_->mark_compact_collector()->marking_deque()->IsEmpty()) {
         if (completion == FORCE_COMPLETION ||