From b2abfc4966dc0c4395d6511b16b11411045c5c98 Mon Sep 17 00:00:00 2001
From: Omer Katz <omerkatz@chromium.org>
Date: Thu, 19 Sep 2024 10:50:09 +0200
Subject: [PATCH] test: remove deprecated v8 flag from nodejs test

test-cli-node-options.js is checking the existance of a
`--huge-max-old-generation-size` V8 flag. That flag was recently
deprected (it currently remains as nop, see crrev.com/c/5831467)
and will soon be completely removed.
---
 doc/api/cli.md                         | 3 ---
 src/node_options.cc                    | 5 -----
 test/parallel/test-cli-node-options.js | 1 -
 3 files changed, 9 deletions(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index fe78ddcfd52749..c554b345bd86e3 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -3096,7 +3096,6 @@ V8 options that are allowed are:
 * `--disallow-code-generation-from-strings`
 * `--enable-etw-stack-walking`
 * `--expose-gc`
-* `--huge-max-old-generation-size`
 * `--interpreted-frames-native-stack`
 * `--jitless`
 * `--max-old-space-size`
@@ -3430,8 +3429,6 @@ documented here:
 
 ### `--harmony-shadow-realm`
 
-### `--huge-max-old-generation-size`
-
 ### `--jitless`
 
 ### `--interpreted-frames-native-stack`
diff --git a/src/node_options.cc b/src/node_options.cc
index cc290661ec3b76..7ee830f52ad044 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -900,11 +900,6 @@ PerIsolateOptionsParser::PerIsolateOptionsParser(
             "disallow eval and friends",
             V8Option{},
             kAllowedInEnvvar);
-  AddOption("--huge-max-old-generation-size",
-            "increase default maximum heap size on machines with 16GB memory "
-            "or more",
-            V8Option{},
-            kAllowedInEnvvar);
   AddOption("--jitless",
             "disable runtime allocation of executable memory",
             V8Option{},
diff --git a/test/parallel/test-cli-node-options.js b/test/parallel/test-cli-node-options.js
index e898a81af09ca6..22d5a342df5d55 100644
--- a/test/parallel/test-cli-node-options.js
+++ b/test/parallel/test-cli-node-options.js
@@ -70,7 +70,6 @@ if (common.hasCrypto) {
 expect('--abort_on-uncaught_exception', 'B\n');
 expect('--disallow-code-generation-from-strings', 'B\n');
 expect('--expose-gc', 'B\n');
-expect('--huge-max-old-generation-size', 'B\n');
 expect('--jitless', 'B\n');
 expect('--max-old-space-size=0', 'B\n');
 expect('--max-semi-space-size=0', 'B\n');