src: update std::vector<v8::Local<T>> to use v8::LocalVector<T>

nodejs/node#57578

Author: codebytere

shell/common/node_bindings.cc

@@ -992,11 +992,10 @@ void OnNodePreload(node::Environment* env,
   }
 
   // Execute lib/node/init.ts.
-  std::vector<v8::Local<v8::String>> bundle_params = {
-      node::FIXED_ONE_BYTE_STRING(env->isolate(), "process"),
-      node::FIXED_ONE_BYTE_STRING(env->isolate(), "require"),
-  };
-  std::vector<v8::Local<v8::Value>> bundle_args = {process, require};
+  v8::LocalVector<v8::String> bundle_params(
+      env->isolate(), {node::FIXED_ONE_BYTE_STRING(env->isolate(), "process"),
+                       node::FIXED_ONE_BYTE_STRING(env->isolate(), "require")});
+  v8::LocalVector<v8::Value> bundle_args(env->isolate(), {process, require});
   electron::util::CompileAndCall(env->context(), "electron/js2c/node_init",
                                  &bundle_params, &bundle_args);
 }

shell/common/node_util.cc

@@ -22,8 +22,8 @@ namespace electron::util {
 v8::MaybeLocal<v8::Value> CompileAndCall(
     v8::Local<v8::Context> context,
     const char* id,
-    std::vector<v8::Local<v8::String>>* parameters,
-    std::vector<v8::Local<v8::Value>>* arguments) {
+    v8::LocalVector<v8::String>* parameters,
+    v8::LocalVector<v8::Value>* arguments) {
   v8::Isolate* isolate = context->GetIsolate();
   v8::TryCatch try_catch(isolate);
 

shell/common/node_util.h

@@ -41,8 +41,8 @@ void EmitWarning(std::string_view warning_msg, std::string_view warning_type);
 v8::MaybeLocal<v8::Value> CompileAndCall(
     v8::Local<v8::Context> context,
     const char* id,
-    std::vector<v8::Local<v8::String>>* parameters,
-    std::vector<v8::Local<v8::Value>>* arguments);
+    v8::LocalVector<v8::String>* parameters,
+    v8::LocalVector<v8::Value>* arguments);
 
 // Wrapper for node::CreateEnvironment that logs failure
 node::Environment* CreateEnvironment(v8::Isolate* isolate,

shell/renderer/electron_sandboxed_renderer_client.cc

@@ -125,10 +125,10 @@ void ElectronSandboxedRendererClient::DidCreateScriptContext(
   auto binding = v8::Object::New(isolate);
   InitializeBindings(binding, context, render_frame);
 
-  std::vector<v8::Local<v8::String>> sandbox_preload_bundle_params = {
-      node::FIXED_ONE_BYTE_STRING(isolate, "binding")};
+  v8::LocalVector<v8::String> sandbox_preload_bundle_params(
+      isolate, {node::FIXED_ONE_BYTE_STRING(isolate, "binding")});
 
-  std::vector<v8::Local<v8::Value>> sandbox_preload_bundle_args = {binding};
+  v8::LocalVector<v8::Value> sandbox_preload_bundle_args(isolate, {binding});
 
   util::CompileAndCall(
       isolate->GetCurrentContext(), "electron/js2c/sandbox_bundle",

shell/renderer/preload_realm_context.cc

@@ -176,10 +176,10 @@ class PreloadRealmLifetimeController
     process.SetReadOnly("type", "service-worker");
     process.SetReadOnly("contextIsolated", true);
 
-    std::vector<v8::Local<v8::String>> preload_realm_bundle_params = {
-        node::FIXED_ONE_BYTE_STRING(isolate, "binding")};
+    v8::LocalVector<v8::String> preload_realm_bundle_params(
+        isolate, {node::FIXED_ONE_BYTE_STRING(isolate, "binding")});
 
-    std::vector<v8::Local<v8::Value>> preload_realm_bundle_args = {binding};
+    v8::LocalVector<v8::Value> preload_realm_bundle_args(isolate, {binding});
 
     util::CompileAndCall(context, "electron/js2c/preload_realm_bundle",
                          &preload_realm_bundle_params,

shell/renderer/renderer_client_base.cc

@@ -597,11 +597,11 @@ void RendererClientBase::SetupMainWorldOverrides(
     }
   }
 
-  std::vector<v8::Local<v8::String>> isolated_bundle_params = {
-      node::FIXED_ONE_BYTE_STRING(isolate, "isolatedApi")};
+  v8::LocalVector<v8::String> isolated_bundle_params(
+      isolate, {node::FIXED_ONE_BYTE_STRING(isolate, "isolatedApi")});
 
-  std::vector<v8::Local<v8::Value>> isolated_bundle_args = {
-      isolated_api.GetHandle()};
+  v8::LocalVector<v8::Value> isolated_bundle_args(isolate,
+                                                  {isolated_api.GetHandle()});
 
   util::CompileAndCall(context, "electron/js2c/isolated_bundle",
                        &isolated_bundle_params, &isolated_bundle_args);