[esnext] Fix various callsites to use is_resumable, not is_generator

src/debug/debug.cc

Index: src/debug/debug.cc
diff --git a/src/debug/debug.cc b/src/debug/debug.cc
index e1f0d489c74e202104f5ba997ef0296638bfdd79..e8d615879040ffd0857721683275011c868db222 100644
--- a/src/debug/debug.cc
+++ b/src/debug/debug.cc
@@ -1329,7 +1329,7 @@ bool Debug::PrepareFunctionForBreakPoints(Handle<SharedFunctionInfo> shared) {
     // smarter here and avoid the heap walk.
     HeapIterator iterator(isolate_->heap());
     HeapObject* obj;
-    bool include_generators = !is_interpreted && shared->is_generator();
+    bool include_generators = !is_interpreted && shared->is_resumable();

[Yang, 2016/05/30 12:32:46]

how about we rename "include_generators" to "find_resumable"?

[Dan Ehrenberg, 2016/05/31 08:53:37]

Good idea, done.

 
     while ((obj = iterator.next())) {
       if (obj->IsJSFunction()) {
@@ -1341,6 +1341,8 @@ bool Debug::PrepareFunctionForBreakPoints(Handle<SharedFunctionInfo> shared) {
         if (is_interpreted) continue;
         if (function->shared() == *shared) functions.Add(handle(function));
       } else if (include_generators && obj->IsJSGeneratorObject()) {
+        // This case handles async functions as well, as they use generator
+        // objects for in-progress async function execution.
         JSGeneratorObject* generator_obj = JSGeneratorObject::cast(obj);
         if (!generator_obj->is_suspended()) continue;
         JSFunction* function = generator_obj->function();