src: avoid std::make_unique

Work around nodejs/build#1254, which
effectively breaks stress test CI and CITGM, by avoiding
`std::make_unique` for now.
This workaround should be reverted once that issue is resolved.
Refs: nodejs/build#1254
PR-URL: #20386
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Matheus Marchini <matheus@sthima.com>

Author: addaleax

src/inspector_agent.cc

@@ -367,8 +367,9 @@ class NodeInspectorClient : public V8InspectorClient {
 intconnectFrontend(std::unique_ptr<InspectorSessionDelegate> delegate) {
  events_dispatched_ = true;
 int session_id = next_session_id_++;
- channels_[session_id] =
- std::make_unique<ChannelImpl>(client_, std::move(delegate));
+// TODO(addaleax): Revert back to using make_unique once we get issues
+// with CI resolved (i.e. revert the patch that added this comment).
+ channels_[session_id].reset(newChannelImpl(client_, std::move(delegate)));
 return session_id;
  }
 
@@ -569,7 +570,8 @@ void Agent::Stop() {
 std::unique_ptr<InspectorSession> Agent::Connect(
  std::unique_ptr<InspectorSessionDelegate> delegate) {
 int session_id = client_->connectFrontend(std::move(delegate));
-return std::make_unique<InspectorSession>(session_id, client_);
+return std::unique_ptr<InspectorSession>(
+newInspectorSession(session_id, client_));
 }
 
 voidAgent::WaitForDisconnect() {

src/inspector_io.cc

@@ -357,8 +357,8 @@ std::vector<std::string> InspectorIo::GetTargetIds() const {
 TransportAction InspectorIo::Attach(int session_id) {
  Agent* agent = parent_env_->inspector_agent();
 fprintf(stderr, "Debugger attached.\n");
- sessions_[session_id] =
-agent->Connect(std::make_unique<IoSessionDelegate>(this, session_id));
+ sessions_[session_id] = agent->Connect(std::unique_ptr<IoSessionDelegate>(
+newIoSessionDelegate(this, session_id)));
 return TransportAction::kAcceptSession;
 }
 

src/inspector_js_api.cc

@@ -66,8 +66,8 @@ class JSBindingsConnection : public AsyncWrap {
  callback_(env->isolate(), callback) {
 Wrap(wrap, this);
  Agent* inspector = env->inspector_agent();
- session_ = inspector->Connect(
-std::make_unique<JSBindingsSessionDelegate>(env, this));
+ session_ = inspector->Connect(std::unique_ptr<JSBindingsSessionDelegate>(
+newJSBindingsSessionDelegate(env, this)));
  }
 
 voidOnMessage(Local<Value> value) {