[inspector] async stacks for Promise.then calls...

src/runtime/runtime-promise.cc

 // Copyright 2016 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.
 #include "src/runtime/runtime-utils.h"
 
 #include "src/debug/debug.h"
 #include "src/elements.h"
 #include "src/promise-utils.h"
 
 namespace v8 {
@@ skipping 51 matching lines @@
                                Handle<Object> deferred, Handle<Object> status) {
   Handle<Object> debug_id = isolate->factory()->undefined_value();
   Handle<Object> debug_name = isolate->factory()->undefined_value();
   if (isolate->debug()->is_active()) {
     MaybeHandle<Object> maybe_result;
     Handle<Object> deferred_obj(deferred);
 
     if (deferred->IsFixedArray()) {
       deferred_obj = isolate->factory()->undefined_value();
     }
-
-    Handle<Object> argv[] = {deferred_obj, status};
+    Handle<Object> argv[] = {deferred_obj, status,
+                             isolate->debug()->NextAsyncTaskId(promise)};
     maybe_result = Execution::TryCall(
         isolate, isolate->promise_debug_get_info(),
         isolate->factory()->undefined_value(), arraysize(argv), argv);
 
     Handle<Object> result;
     if ((maybe_result).ToHandle(&result)) {
       CHECK(result->IsJSArray());
       Handle<JSArray> array = Handle<JSArray>::cast(result);
       ElementsAccessor* accessor = array->GetElementsAccessor();
       DCHECK(accessor->HasElement(array, 0));
@@ skipping 13 matching lines @@
                 Handle<Object> result) {
   promise->set_status(status);
   promise->set_result(*result);
   promise->set_deferred(isolate->heap()->undefined_value());
   promise->set_fulfill_reactions(isolate->heap()->undefined_value());
   promise->set_reject_reactions(isolate->heap()->undefined_value());
 }
 
 void PromiseFulfill(Isolate* isolate, Handle<JSPromise> promise,
                     Handle<Smi> status, Handle<Object> value) {
+  if (isolate->debug()->is_active()) {
+    isolate->debug()->OnAsyncTaskEvent(
+        debug::EnqueueRecurring, isolate->debug()->NextAsyncTaskId(promise),
+        status->value() == kPromiseFulfilled
+            ? isolate->factory()->PromiseResolve_string()
+            : isolate->factory()->PromiseReject_string());
+  }
+
   // Check if there are any callbacks.
   if (!promise->deferred()->IsUndefined(isolate)) {
     Handle<Object> tasks((status->value() == kPromiseFulfilled)
                              ? promise->fulfill_reactions()
                              : promise->reject_reactions(),
                          isolate);
     Handle<Object> deferred(promise->deferred(), isolate);
     EnqueuePromiseReactionJob(isolate, promise, value, tasks, deferred, status);
   }
 
@@ skipping 47 matching lines @@
 
   // TODO(gsathya): Add fast path for native promises with unmodified
   // PromiseThen (which don't need these resolving functions, but
   // instead can just call resolve/reject directly).
   Handle<JSFunction> resolve, reject;
   PromiseUtils::CreateResolvingFunctions(
       isolate, promise, isolate->factory()->false_value(), &resolve, &reject);
 
   Handle<Object> debug_id, debug_name;
   if (isolate->debug()->is_active()) {
-    debug_id =
-        handle(Smi::FromInt(isolate->GetNextDebugMicrotaskId()), isolate);
+    debug_id = isolate->debug()->NextAsyncTaskId(promise);
     debug_name = isolate->factory()->PromiseResolveThenableJob_string();
-    isolate->debug()->OnAsyncTaskEvent(isolate->factory()->enqueue_string(),
-                                       debug_id,
-                                       Handle<String>::cast(debug_name));
   } else {
     debug_id = isolate->factory()->undefined_value();
     debug_name = isolate->factory()->undefined_value();
   }
 
   Handle<PromiseResolveThenableJobInfo> info =
       isolate->factory()->NewPromiseResolveThenableJobInfo(
           resolution, then, resolve, reject, debug_id, debug_name,
           isolate->native_context());
   isolate->EnqueueMicrotask(info);
@@ skipping 89 matching lines @@
   HandleScope scope(isolate);
   DCHECK(args.length() == 1);
   CONVERT_ARG_HANDLE_CHECKED(JSPromise, promise, 0);
 
   promise->set_has_handler(true);
   return isolate->heap()->undefined_value();
 }
 
 }  // namespace internal
 }  // namespace v8