Debugger: prepare code for debugging on a per-function basis.

src/heap/objects-visiting-inl.h

Index: src/heap/objects-visiting-inl.h
diff --git a/src/heap/objects-visiting-inl.h b/src/heap/objects-visiting-inl.h
index e3e8e710203ea03bdb376d97a03d6ba6722326bb..6bec9e53a3e7a915a284a1d4c6dcbe3f247504b0 100644
--- a/src/heap/objects-visiting-inl.h
+++ b/src/heap/objects-visiting-inl.h
@@ -659,6 +659,11 @@ bool StaticMarkingVisitor<StaticVisitor>::IsFlushable(
     return false;
   }
 
+  // Function is subject to debugging, do not flush.
+  if (shared_info->HasDebugInfo()) {

[Michael Starzinger, 2015/07/20 11:33:41]

Ouch, this is scary! Is this a hard requirement? If it is then we need to make
double-sure that flushing decisions are revisited whenever this predicate
changes. Otherwise the SharedFunctionInfo might be enqueued as a flushing
candidate already when the debugger adds breakpoints.

[Yang, 2015/07/20 12:13:47]

Done.

+    return false;
+  }
+
   // The function must be compiled and have the source code available,
   // to be able to recompile it in case we need the function again.
   if (!(shared_info->is_compiled() && HasSourceCode(heap, shared_info))) {