Pass StackTrace directly to Hermes ConsoleMessage

Summary:
Hermes' ConsoleMessage constructor now accepts StackTrace, so the construction can be done in one go.

Changelog: [Internal]

Reviewed By: motiz88

Differential Revision: D56738060

fbshipit-source-id: 709b47d8f9cf69994e4c5eaa4f9310e70a4d9ed0

Author: dannysu

packages/react-native/ReactCommon/hermes/inspector-modern/chrome/HermesRuntimeTargetDelegate.cpp

@@ -143,10 +143,10 @@ class HermesRuntimeTargetDelegate::Impl final : public RuntimeTargetDelegate {
       hermesStackTrace = std::move(**hermesStackTraceWrapper);
     }
     HermesConsoleMessage hermesConsoleMessage{
-        message.timestamp, type, std::move(message.args)};
-    // NOTE: HermesConsoleMessage should really have a constructor that takes a
-    // stack trace.
-    hermesConsoleMessage.stackTrace = std::move(hermesStackTrace);
+        message.timestamp,
+        type,
+        std::move(message.args),
+        std::move(hermesStackTrace)};
     cdpDebugAPI_->addConsoleMessage(std::move(hermesConsoleMessage));
   }