No-op by default (#54335)

Summary:

# Changelog: [Internal]

Actually gate the task scheduling logic, so that the implementation is a no-op by default, if we are not tracing in the background or have Runtime CDP domain enabled.

The drawback is that we might loose stack traces for some component renders, if the corresponding JSX declaration was called before we've enabled the `console.createTask()`.

Reviewed By: huntie

Differential Revision: D85849862

Author: hoxyq

packages/react-native/ReactCommon/jsinspector-modern/RuntimeTargetConsole.cpp

@@ -555,7 +555,7 @@ jsi::Value consoleCreateTask(
     const jsi::Value* args,
     size_t count,
     RuntimeTargetDelegate& runtimeTargetDelegate,
-    bool /* enabled */) {
+    bool enabled) {
   if (count < 1 || !args[0].isString()) {
     throw JSError(runtime, "First argument must be a non-empty string");
   }
@@ -565,9 +565,12 @@ jsi::Value consoleCreateTask(
   }
 
   jsi::Object task{runtime};
-  auto taskContext = std::make_shared<ConsoleTaskContext>(
-      runtime, runtimeTargetDelegate, name);
-  taskContext->schedule();
+  std::shared_ptr<ConsoleTaskContext> taskContext = nullptr;
+  if (enabled) {
+    taskContext = std::make_shared<ConsoleTaskContext>(
+        runtime, runtimeTargetDelegate, name);
+    taskContext->schedule();
+  }
 
   task.setProperty(
       runtime,