[inspector] introduced debug::SetAsyncTaskListener

src/api.cc

Index: src/api.cc
diff --git a/src/api.cc b/src/api.cc
index e313812a6ceafc3adc70cebe1d00b3c663967c21..927ff012f7740e4afd3a4c3e6f837f2fb4e68787 100644
--- a/src/api.cc
+++ b/src/api.cc
@@ -9308,6 +9308,14 @@ MaybeLocal<UnboundScript> debug::CompileInspectorScript(Isolate* v8_isolate,
   RETURN_ESCAPED(ToApiHandle<UnboundScript>(result));
 }
 
+void debug::SetAsyncTaskListener(Isolate* v8_isolate,

[jgruber, 2017/01/13 16:38:38]

Just a general question - what about moving debugging API code to another file?
Was there a reason that it needs to be in here?

[kozy, 2017/01/13 17:05:08]

This file contains a lot of useful macros for api, we probably can extract this
macros into separate header and then put debug-interface implementation into
separate file.
I'm strongly support extracting of this because any change of this files force
half of V8 recompilation.

+                                 debug::AsyncTaskListener listener,
+                                 void* data) {
+  i::Isolate* isolate = reinterpret_cast<i::Isolate*>(v8_isolate);
+  ENTER_V8(isolate);
+  isolate->debug()->SetAsyncTaskListener(listener, data);
+}
+
 Local<String> CpuProfileNode::GetFunctionName() const {
   const i::ProfileNode* node = reinterpret_cast<const i::ProfileNode*>(this);
   i::Isolate* isolate = node->isolate();