From 4212a65498659f92f196569ee20e6ecf5d191fb4 Mon Sep 17 00:00:00 2001
From: Joyee Cheung <joyeec9h3@gmail.com>
Date: Fri, 19 Apr 2019 17:15:04 +0800
Subject: [PATCH 1/4] inspector: split --cpu-prof-path to --prof-dir and
 --cpu-prof-name

To improve the integration of `--cpu-prof` with workers, this patch
splits `--cpu-prof-path` into `--prof-dir` and `--cpu-prof-name`,
so when a worker is launched from a thread that enables
`--cpu-prof`, if the parent thread sets `--prof-dir`, then the
profile of both thread would be generated to the specified directory.
If they end up specifying the same `--cpu-prof-name` the behavior
is undefined the last profile will overwritten the first one.
---
 doc/api/cli.md                                |  31 ++--
 doc/node.1                                    |  18 ++-
 src/env-inl.h                                 |  31 ++++
 src/env.h                                     |   6 +
 src/inspector_profiler.cc                     |  16 +-
 src/node.cc                                   |   3 +-
 src/node_options.cc                           |  22 ++-
 src/node_options.h                            |   3 +-
 .../workload/fibonacci-worker-argv.js         |   7 +
 test/fixtures/workload/fibonacci-worker.js    |   4 +-
 test/sequential/test-cpu-prof.js              | 147 +++++++++++++++---
 11 files changed, 233 insertions(+), 55 deletions(-)
 create mode 100644 test/fixtures/workload/fibonacci-worker-argv.js

diff --git a/doc/api/cli.md b/doc/api/cli.md
index c26177444c7894..23d2642cee575b 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -83,8 +83,13 @@ added: REPLACEME
 > Stability: 1 - Experimental
 
 Starts the V8 CPU profiler on start up, and writes the CPU profile to disk
-before exit. If `--cpu-prof-path` is not specified, the profile will be
-written to `${cwd}/CPU.${yyyymmdd}.${hhmmss}.${pid}.${tid}.${seq}.cpuprofile`.
+before exit.
+
+If `--prof-dir` is not specified, the generated profile will be placed in the
+current working directory.
+
+If `--cpu-prof-name` is not specified, the generated profile will be
+named `CPU.${yyyymmdd}.${hhmmss}.${pid}.${tid}.${seq}.cpuprofile`.
 
 ```console
 $ node --cpu-prof index.js
@@ -92,19 +97,14 @@ $ ls *.cpuprofile
 CPU.20190409.202950.15293.0.0.cpuprofile
 ```
 
-### `--cpu-prof-path`
+### `--cpu-prof-name`
 <!-- YAML
 added: REPLACEME
 -->
 
 > Stability: 1 - Experimental
 
-Location where the CPU profile generated by `--cpu-prof`
-should be written to. When used alone, it implies `--cpu-prof`.
-
-```console
-$ node --cpu-prof-path /tmp/test.cpuprofile index.js
-```
+Specify the file name of the CPU profile generated by `--cpu-prof`.
 
 ### `--diagnostic-report-directory=directory`
 <!-- YAML
@@ -475,6 +475,19 @@ added: v2.0.0
 
 Generate V8 profiler output.
 
+### `--prof-dir`
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1 - Experimental
+
+Specify the directory where the CPU profiles generated by `--cpu-prof` will
+be placed.
+
+This does not apply to `--prof` whose output is handled by V8 instead of
+Node.js.
+
 ### `--prof-process`
 <!-- YAML
 added: v5.2.0
diff --git a/doc/node.1 b/doc/node.1
index 8387b6282a9b8d..455641584e79e6 100644
--- a/doc/node.1
+++ b/doc/node.1
@@ -81,13 +81,12 @@ Print source-able bash completion script for Node.js.
 .It Fl -cpu-prof
 Start the V8 CPU profiler on start up, and write the CPU profile to disk
 before exit. If
-.Fl -cpu-prof-path
-is not specified, the profile will be written to the current working directory.
+.Fl -prof-dir
+is not specified, the profile will be written to the current working directory
+with a generated file name.
 .
-.It Fl -cpu-prof-path
-Path the V8 CPU profile generated with
-.Fl -cpu-prof
-will be written to. When used alone, it implies
+.It Fl -cpu-prof-name
+File name of the V8 CPU profile generated with
 .Fl -cpu-prof
 .
 .It Fl -diagnostic-report-directory
@@ -241,6 +240,13 @@ Instructs the module loader to preserve symbolic links when resolving and cachin
 .It Fl -prof
 Generate V8 profiler output.
 .
+.It Fl -prof-dir
+The directory where the CPU profiles generated by
+.Fl -cpu-prof
+will be placed. Note that this does not apply to
+.Fl -prof
+whose output is handled by V8, not Node.js.
+.
 .It Fl -prof-process
 Process V8 profiler output generated using the V8 option
 .Fl -prof .
diff --git a/src/env-inl.h b/src/env-inl.h
index 3a6caa19f86d6b..04cc6d2fe2721b 100644
--- a/src/env-inl.h
+++ b/src/env-inl.h
@@ -38,6 +38,14 @@
 
 #include <utility>
 
+#ifdef _WIN32
+/* MAX_PATH is in characters, not bytes. Make sure we have enough headroom. */
+#define CWD_BUFSIZE (MAX_PATH * 4)
+#else
+#include <climits>  // PATH_MAX on Solaris.
+#define CWD_BUFSIZE (PATH_MAX)
+#endif
+
 namespace node {
 
 inline v8::Isolate* IsolateData::isolate() const {
@@ -678,6 +686,29 @@ inline void Environment::set_cpu_profile_path(const std::string& path) {
 inline const std::string& Environment::cpu_profile_path() const {
   return cpu_profile_path_;
 }
+
+inline void Environment::set_prof_dir(const std::string& path) {
+  prof_dir_ = path;
+}
+
+inline const std::string& Environment::prof_dir() const {
+  return prof_dir_;
+}
+
+inline void Environment::InitializeProfDir(const std::string& dir) {
+  if (!dir.empty()) {
+    prof_dir_ = dir;
+    return;
+  }
+  char cwd[CWD_BUFSIZE];
+  size_t size = CWD_BUFSIZE;
+  int err = uv_cwd(cwd, &size);
+  // TODO(joyeecheung): fallback to exec path / argv[0]
+  CHECK_EQ(err, 0);
+  CHECK_GT(size, 0);
+  prof_dir_ = cwd;
+}
+
 #endif  // HAVE_INSPECTOR
 
 inline std::shared_ptr<HostPort> Environment::inspector_host_port() {
diff --git a/src/env.h b/src/env.h
index cfef54dcbd7b3e..d16361c03c25ac 100644
--- a/src/env.h
+++ b/src/env.h
@@ -1137,6 +1137,11 @@ class Environment : public MemoryRetainer {
 
   inline void set_cpu_profile_path(const std::string& path);
   inline const std::string& cpu_profile_path() const;
+
+  inline void set_prof_dir(const std::string& path);
+  inline const std::string& prof_dir() const;
+
+  inline void InitializeProfDir(const std::string& dir);
 #endif  // HAVE_INSPECTOR
 
  private:
@@ -1173,6 +1178,7 @@ class Environment : public MemoryRetainer {
   std::unique_ptr<profiler::V8CoverageConnection> coverage_connection_;
   std::unique_ptr<profiler::V8CpuProfilerConnection> cpu_profiler_connection_;
   std::string coverage_directory_;
+  std::string prof_dir_;
   std::string cpu_profile_path_;
 #endif  // HAVE_INSPECTOR
 
diff --git a/src/inspector_profiler.cc b/src/inspector_profiler.cc
index 46d4c4fec8d19e..e33a2adcbb8841 100644
--- a/src/inspector_profiler.cc
+++ b/src/inspector_profiler.cc
@@ -318,19 +318,13 @@ void StartCoverageCollection(Environment* env) {
   env->coverage_connection()->Start();
 }
 
-void StartCpuProfiling(Environment* env, const std::string& profile_path) {
-  std::string path;
-  if (profile_path.empty()) {
-    char cwd[CWD_BUFSIZE];
-    size_t size = CWD_BUFSIZE;
-    int err = uv_cwd(cwd, &size);
-    // TODO(joyeecheung): fallback to exec path / argv[0]
-    CHECK_EQ(err, 0);
-    CHECK_GT(size, 0);
+void StartCpuProfiling(Environment* env, const std::string& profile_name) {
+  std::string path = env->prof_dir() + std::string(kPathSeparator);
+  if (profile_name.empty()) {
     DiagnosticFilename filename(env, "CPU", "cpuprofile");
-    path = cwd + std::string(kPathSeparator) + (*filename);
+    path += *filename;
   } else {
-    path = profile_path;
+    path += profile_name;
   }
   env->set_cpu_profile_path(std::move(path));
   env->set_cpu_profiler_connection(
diff --git a/src/node.cc b/src/node.cc
index 9cdc4e5e2a4f8a..36132f2f5b98db 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -239,7 +239,8 @@ MaybeLocal<Value> RunBootstrapping(Environment* env) {
 
 #if HAVE_INSPECTOR
   if (env->options()->cpu_prof) {
-    profiler::StartCpuProfiling(env, env->options()->cpu_prof_path);
+    env->InitializeProfDir(env->options()->prof_dir);
+    profiler::StartCpuProfiling(env, env->options()->cpu_prof_name);
   }
 #endif  // HAVE_INSPECTOR
 
diff --git a/src/node_options.cc b/src/node_options.cc
index db92a0c2fd1112..49b8eb0bad07d4 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -149,6 +149,15 @@ void EnvironmentOptions::CheckOptions(std::vector<std::string>* errors) {
   }
 
 #if HAVE_INSPECTOR
+  if (!cpu_prof && !cpu_prof_name.empty()) {
+    errors->push_back("--cpu-prof-name must be used with --cpu-prof");
+  }
+
+  // We may add support for other profiles in the future.
+  if (!prof_dir.empty() && !cpu_prof) {
+    errors->push_back("--prof-dir must be used with --cpu-prof");
+  }
+
   debug_options_.CheckOptions(errors);
 #endif  // HAVE_INSPECTOR
 }
@@ -338,11 +347,14 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "to disk before exit. If --cpu-prof-path is not specified, write "
             "the profile to the current working directory.",
             &EnvironmentOptions::cpu_prof);
-  AddOption("--cpu-prof-path",
-            "Path the V8 CPU profile generated with --cpu-prof will be "
-            "written to.",
-            &EnvironmentOptions::cpu_prof_path);
-  Implies("--cpu-prof-path", "--cpu-prof");
+  AddOption("--cpu-prof-name",
+            "specified file name of the V8 CPU profile generated with "
+            "--cpu-prof",
+            &EnvironmentOptions::cpu_prof_name);
+  AddOption("--prof-dir",
+            "Directory where the V8 profiles generated by --cpu-prof will be "
+            "placed. Does not affect --prof.",
+            &EnvironmentOptions::prof_dir);
 #endif  // HAVE_INSPECTOR
   AddOption("--redirect-warnings",
             "write warnings to file instead of stderr",
diff --git a/src/node_options.h b/src/node_options.h
index d0d35b162e6ea6..58cc8d502f2492 100644
--- a/src/node_options.h
+++ b/src/node_options.h
@@ -110,7 +110,8 @@ class EnvironmentOptions : public Options {
   bool preserve_symlinks_main = false;
   bool prof_process = false;
 #if HAVE_INSPECTOR
-  std::string cpu_prof_path;
+  std::string prof_dir;
+  std::string cpu_prof_name;
   bool cpu_prof = false;
 #endif  // HAVE_INSPECTOR
   std::string redirect_warnings;
diff --git a/test/fixtures/workload/fibonacci-worker-argv.js b/test/fixtures/workload/fibonacci-worker-argv.js
new file mode 100644
index 00000000000000..9c968bfeec4ae2
--- /dev/null
+++ b/test/fixtures/workload/fibonacci-worker-argv.js
@@ -0,0 +1,7 @@
+'use strict';
+
+const { Worker } = require('worker_threads');
+const path = require('path');
+new Worker(path.join(__dirname, 'fibonacci.js'), {
+  execArgv: ['--cpu-prof']
+});
diff --git a/test/fixtures/workload/fibonacci-worker.js b/test/fixtures/workload/fibonacci-worker.js
index 9c968bfeec4ae2..d92fb169317941 100644
--- a/test/fixtures/workload/fibonacci-worker.js
+++ b/test/fixtures/workload/fibonacci-worker.js
@@ -2,6 +2,4 @@
 
 const { Worker } = require('worker_threads');
 const path = require('path');
-new Worker(path.join(__dirname, 'fibonacci.js'), {
-  execArgv: ['--cpu-prof']
-});
+new Worker(path.join(__dirname, 'fibonacci.js'));
diff --git a/test/sequential/test-cpu-prof.js b/test/sequential/test-cpu-prof.js
index 8b9001ba674e23..869982c1f72996 100644
--- a/test/sequential/test-cpu-prof.js
+++ b/test/sequential/test-cpu-prof.js
@@ -1,6 +1,6 @@
 'use strict';
 
-// This tests that --cpu-prof and --cpu-prof-path works.
+// This tests that --cpu-prof, --prof-dir and --cpu-prof-name works.
 
 const common = require('../common');
 if (process.features.debug &&
@@ -27,17 +27,22 @@ function getCpuProfiles(dir) {
     .map((file) => path.join(dir, file));
 }
 
-function verifyFrames(output, file, suffix) {
+function getFrames(output, file, suffix) {
   const data = fs.readFileSync(file, 'utf8');
   const profile = JSON.parse(data);
   const frames = profile.nodes.filter((i) => {
     const frame = i.callFrame;
     return frame.url.endsWith(suffix);
   });
+  return { frames, nodes: profile.nodes };
+}
+
+function verifyFrames(output, file, suffix) {
+  const { frames, nodes } = getFrames(output, file, suffix);
   if (frames.length === 0) {
     // Show native debug output and the profile for debugging.
     console.log(output.stderr.toString());
-    console.log(profile.nodes);
+    console.log(nodes);
   }
   assert.notDeepStrictEqual(frames, []);
 }
@@ -118,11 +123,11 @@ const env = {
   verifyFrames(output, profiles[0], 'fibonacci-sigint.js');
 }
 
-// Outputs CPU profile from worker.
+// Outputs CPU profile from worker when execArgv is set.
 {
   tmpdir.refresh();
   const output = spawnSync(process.execPath, [
-    fixtures.path('workload', 'fibonacci-worker.js'),
+    fixtures.path('workload', 'fibonacci-worker-argv.js'),
   ], {
     cwd: tmpdir.path,
     env
@@ -136,13 +141,56 @@ const env = {
   verifyFrames(output, profiles[0], 'fibonacci.js');
 }
 
-// Output to specified --cpu-prof-path without --cpu-prof
+// --cpu-prof-name without --cpu-prof
+{
+  tmpdir.refresh();
+  const output = spawnSync(process.execPath, [
+    '--cpu-prof-name',
+    'test.cpuprofile',
+    fixtures.path('workload', 'fibonacci.js'),
+  ], {
+    cwd: tmpdir.path,
+    env
+  });
+  const stderr = output.stderr.toString().trim();
+  if (output.status !== 9) {
+    console.log(stderr);
+  }
+  assert.strictEqual(output.status, 9);
+  assert.strictEqual(
+    stderr,
+    `${process.execPath}: --cpu-prof-name must be used with --cpu-prof`);
+}
+
+// --prof-dir without --cpu-prof
+{
+  tmpdir.refresh();
+  const output = spawnSync(process.execPath, [
+    '--prof-dir',
+    'prof',
+    fixtures.path('workload', 'fibonacci.js'),
+  ], {
+    cwd: tmpdir.path,
+    env
+  });
+  const stderr = output.stderr.toString().trim();
+  if (output.status !== 9) {
+    console.log(stderr);
+  }
+  assert.strictEqual(output.status, 9);
+  assert.strictEqual(
+    stderr,
+    `${process.execPath}: --prof-dir must be used with --cpu-prof`);
+}
+
+// --cpu-prof-name
 {
   tmpdir.refresh();
   const file = path.join(tmpdir.path, 'test.cpuprofile');
   const output = spawnSync(process.execPath, [
-    '--cpu-prof-path',
-    file,
+    '--cpu-prof',
+    '--cpu-prof-name',
+    'test.cpuprofile',
     fixtures.path('workload', 'fibonacci.js'),
   ], {
     cwd: tmpdir.path,
@@ -157,14 +205,13 @@ const env = {
   verifyFrames(output, file, 'fibonacci.js');
 }
 
-// Output to specified --cpu-prof-path with --cpu-prof
+// relative --prof-dir
 {
   tmpdir.refresh();
-  const file = path.join(tmpdir.path, 'test.cpuprofile');
   const output = spawnSync(process.execPath, [
     '--cpu-prof',
-    '--cpu-prof-path',
-    file,
+    '--prof-dir',
+    'prof',
     fixtures.path('workload', 'fibonacci.js'),
   ], {
     cwd: tmpdir.path,
@@ -174,19 +221,47 @@ const env = {
     console.log(output.stderr.toString());
   }
   assert.strictEqual(output.status, 0);
-  const profiles = getCpuProfiles(tmpdir.path);
-  assert.deepStrictEqual(profiles, [file]);
-  verifyFrames(output, file, 'fibonacci.js');
+  const dir = path.join(tmpdir.path, 'prof');
+  assert(fs.existsSync(dir));
+  const profiles = getCpuProfiles(dir);
+  assert.strictEqual(profiles.length, 1);
+  verifyFrames(output, profiles[0], 'fibonacci.js');
 }
 
-// Output to specified --cpu-prof-path when it's not absolute
+// absolute --prof-dir
 {
   tmpdir.refresh();
-  const file = path.join(tmpdir.path, 'test.cpuprofile');
+  const dir = path.join(tmpdir.path, 'prof');
   const output = spawnSync(process.execPath, [
     '--cpu-prof',
-    '--cpu-prof-path',
+    '--prof-dir',
+    dir,
+    fixtures.path('workload', 'fibonacci.js'),
+  ], {
+    cwd: tmpdir.path,
+    env
+  });
+  if (output.status !== 0) {
+    console.log(output.stderr.toString());
+  }
+  assert.strictEqual(output.status, 0);
+  assert(fs.existsSync(dir));
+  const profiles = getCpuProfiles(dir);
+  assert.strictEqual(profiles.length, 1);
+  verifyFrames(output, profiles[0], 'fibonacci.js');
+}
+
+// --prof-dir and --cpu-prof-name
+{
+  tmpdir.refresh();
+  const dir = path.join(tmpdir.path, 'prof');
+  const file = path.join(dir, 'test.cpuprofile');
+  const output = spawnSync(process.execPath, [
+    '--cpu-prof',
+    '--cpu-prof-name',
     'test.cpuprofile',
+    '--prof-dir',
+    dir,
     fixtures.path('workload', 'fibonacci.js'),
   ], {
     cwd: tmpdir.path,
@@ -196,7 +271,41 @@ const env = {
     console.log(output.stderr.toString());
   }
   assert.strictEqual(output.status, 0);
-  const profiles = getCpuProfiles(tmpdir.path);
+  assert(fs.existsSync(dir));
+  const profiles = getCpuProfiles(dir);
   assert.deepStrictEqual(profiles, [file]);
   verifyFrames(output, file, 'fibonacci.js');
 }
+
+// --prof-dir with worker
+{
+  tmpdir.refresh();
+  const output = spawnSync(process.execPath, [
+    '--prof-dir',
+    'prof',
+    '--cpu-prof',
+    fixtures.path('workload', 'fibonacci-worker.js'),
+  ], {
+    cwd: tmpdir.path,
+    env
+  });
+  if (output.status !== 0) {
+    console.log(output.stderr.toString());
+  }
+  assert.strictEqual(output.status, 0);
+  const dir = path.join(tmpdir.path, 'prof');
+  assert(fs.existsSync(dir));
+  const profiles = getCpuProfiles(dir);
+  assert.strictEqual(profiles.length, 2);
+  const profile1 = getFrames(output, profiles[0], 'fibonacci.js');
+  const profile2 = getFrames(output, profiles[1], 'fibonacci.js');
+  if (profile1.frames.length === 0 && profile2.frames.length === 0) {
+    // Show native debug output and the profile for debugging.
+    console.log(output.stderr.toString());
+    console.log('CPU path: ', profiles[0]);
+    console.log(profile1.nodes);
+    console.log('CPU path: ', profiles[1]);
+    console.log(profile2.nodes);
+  }
+  assert(profile1.frames.length > 0 || profile2.frames.length > 0);
+}

From f0de081d3c7229ab44b5b3d8f19ae67dccdd800d Mon Sep 17 00:00:00 2001
From: Joyee Cheung <joyeec9h3@gmail.com>
Date: Fri, 19 Apr 2019 22:32:06 +0800
Subject: [PATCH 2/4] squash! inspector: split --cpu-prof-path to --prof-dir
 and --cpu-prof-name

---
 doc/api/cli.md                   | 30 +++++++++++++++---------------
 doc/node.1                       | 16 ++++++++--------
 src/env-inl.h                    | 22 ++++------------------
 src/env.cc                       | 18 ++++++++++++++++++
 src/env.h                        |  8 ++++----
 src/inspector_profiler.cc        |  2 +-
 src/node.cc                      |  2 +-
 src/node_options.cc              |  8 ++++----
 src/node_options.h               |  2 +-
 test/sequential/test-cpu-prof.js | 24 ++++++++++++------------
 10 files changed, 68 insertions(+), 64 deletions(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index 23d2642cee575b..d01f967ea2897b 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -85,8 +85,8 @@ added: REPLACEME
 Starts the V8 CPU profiler on start up, and writes the CPU profile to disk
 before exit.
 
-If `--prof-dir` is not specified, the generated profile will be placed in the
-current working directory.
+If `--node-prof-dir` is not specified, the generated profile will be placed
+in the current working directory.
 
 If `--cpu-prof-name` is not specified, the generated profile will be
 named `CPU.${yyyymmdd}.${hhmmss}.${pid}.${tid}.${seq}.cpuprofile`.
@@ -360,6 +360,19 @@ added: v7.10.0
 
 This option is a no-op. It is kept for compatibility.
 
+### `--node-prof-dir`
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1 - Experimental
+
+Specify the directory where the CPU profiles generated by `--cpu-prof` will
+be placed.
+
+This does not apply to `--prof` whose output is handled by V8 instead of
+Node.js.
+
 ### `--no-deprecation`
 <!-- YAML
 added: v0.8.0
@@ -475,19 +488,6 @@ added: v2.0.0
 
 Generate V8 profiler output.
 
-### `--prof-dir`
-<!-- YAML
-added: REPLACEME
--->
-
-> Stability: 1 - Experimental
-
-Specify the directory where the CPU profiles generated by `--cpu-prof` will
-be placed.
-
-This does not apply to `--prof` whose output is handled by V8 instead of
-Node.js.
-
 ### `--prof-process`
 <!-- YAML
 added: v5.2.0
diff --git a/doc/node.1 b/doc/node.1
index 455641584e79e6..74538002dd4bea 100644
--- a/doc/node.1
+++ b/doc/node.1
@@ -81,7 +81,7 @@ Print source-able bash completion script for Node.js.
 .It Fl -cpu-prof
 Start the V8 CPU profiler on start up, and write the CPU profile to disk
 before exit. If
-.Fl -prof-dir
+.Fl -node-prof-dir
 is not specified, the profile will be written to the current working directory
 with a generated file name.
 .
@@ -213,6 +213,13 @@ Specify the maximum size of HTTP headers in bytes. Defaults to 8KB.
 This option is a no-op.
 It is kept for compatibility.
 .
+.It Fl -node-prof-dir
+The directory where the CPU profiles generated by
+.Fl -cpu-prof
+will be placed. Note that this does not apply to
+.Fl -prof
+whose output is handled by V8, not Node.js.
+.
 .It Fl -no-deprecation
 Silence deprecation warnings.
 .
@@ -240,13 +247,6 @@ Instructs the module loader to preserve symbolic links when resolving and cachin
 .It Fl -prof
 Generate V8 profiler output.
 .
-.It Fl -prof-dir
-The directory where the CPU profiles generated by
-.Fl -cpu-prof
-will be placed. Note that this does not apply to
-.Fl -prof
-whose output is handled by V8, not Node.js.
-.
 .It Fl -prof-process
 Process V8 profiler output generated using the V8 option
 .Fl -prof .
diff --git a/src/env-inl.h b/src/env-inl.h
index 04cc6d2fe2721b..feae4c26b0dd05 100644
--- a/src/env-inl.h
+++ b/src/env-inl.h
@@ -687,26 +687,12 @@ inline const std::string& Environment::cpu_profile_path() const {
   return cpu_profile_path_;
 }
 
-inline void Environment::set_prof_dir(const std::string& path) {
-  prof_dir_ = path;
+inline void Environment::set_node_prof_dir(const std::string& path) {
+  node_prof_dir_ = path;
 }
 
-inline const std::string& Environment::prof_dir() const {
-  return prof_dir_;
-}
-
-inline void Environment::InitializeProfDir(const std::string& dir) {
-  if (!dir.empty()) {
-    prof_dir_ = dir;
-    return;
-  }
-  char cwd[CWD_BUFSIZE];
-  size_t size = CWD_BUFSIZE;
-  int err = uv_cwd(cwd, &size);
-  // TODO(joyeecheung): fallback to exec path / argv[0]
-  CHECK_EQ(err, 0);
-  CHECK_GT(size, 0);
-  prof_dir_ = cwd;
+inline const std::string& Environment::node_prof_dir() const {
+  return node_prof_dir_;
 }
 
 #endif  // HAVE_INSPECTOR
diff --git a/src/env.cc b/src/env.cc
index ccb3c404f91ba2..e705d74de43d11 100644
--- a/src/env.cc
+++ b/src/env.cc
@@ -845,6 +845,24 @@ void Environment::stop_sub_worker_contexts() {
   }
 }
 
+#if HAVE_INSPECTOR
+
+void Environment::InitializeNodeProfDir(const std::string& dir) {
+  if (!dir.empty()) {
+    node_prof_dir_ = dir;
+    return;
+  }
+  char cwd[CWD_BUFSIZE];
+  size_t size = CWD_BUFSIZE;
+  int err = uv_cwd(cwd, &size);
+  // TODO(joyeecheung): fallback to exec path / argv[0]
+  CHECK_EQ(err, 0);
+  CHECK_GT(size, 0);
+  node_prof_dir_ = cwd;
+}
+
+#endif  // HAVE_INSPECTOR
+
 void MemoryTracker::TrackField(const char* edge_name,
                                const CleanupHookCallback& value,
                                const char* node_name) {
diff --git a/src/env.h b/src/env.h
index d16361c03c25ac..b8606f95fe8060 100644
--- a/src/env.h
+++ b/src/env.h
@@ -1138,10 +1138,10 @@ class Environment : public MemoryRetainer {
   inline void set_cpu_profile_path(const std::string& path);
   inline const std::string& cpu_profile_path() const;
 
-  inline void set_prof_dir(const std::string& path);
-  inline const std::string& prof_dir() const;
+  inline void set_node_prof_dir(const std::string& path);
+  inline const std::string& node_prof_dir() const;
 
-  inline void InitializeProfDir(const std::string& dir);
+  void InitializeNodeProfDir(const std::string& dir);
 #endif  // HAVE_INSPECTOR
 
  private:
@@ -1178,7 +1178,7 @@ class Environment : public MemoryRetainer {
   std::unique_ptr<profiler::V8CoverageConnection> coverage_connection_;
   std::unique_ptr<profiler::V8CpuProfilerConnection> cpu_profiler_connection_;
   std::string coverage_directory_;
-  std::string prof_dir_;
+  std::string node_prof_dir_;
   std::string cpu_profile_path_;
 #endif  // HAVE_INSPECTOR
 
diff --git a/src/inspector_profiler.cc b/src/inspector_profiler.cc
index e33a2adcbb8841..6fc02b3d8911cc 100644
--- a/src/inspector_profiler.cc
+++ b/src/inspector_profiler.cc
@@ -319,7 +319,7 @@ void StartCoverageCollection(Environment* env) {
 }
 
 void StartCpuProfiling(Environment* env, const std::string& profile_name) {
-  std::string path = env->prof_dir() + std::string(kPathSeparator);
+  std::string path = env->node_prof_dir() + std::string(kPathSeparator);
   if (profile_name.empty()) {
     DiagnosticFilename filename(env, "CPU", "cpuprofile");
     path += *filename;
diff --git a/src/node.cc b/src/node.cc
index 36132f2f5b98db..8c4e6051262175 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -239,7 +239,7 @@ MaybeLocal<Value> RunBootstrapping(Environment* env) {
 
 #if HAVE_INSPECTOR
   if (env->options()->cpu_prof) {
-    env->InitializeProfDir(env->options()->prof_dir);
+    env->InitializeNodeProfDir(env->options()->node_prof_dir);
     profiler::StartCpuProfiling(env, env->options()->cpu_prof_name);
   }
 #endif  // HAVE_INSPECTOR
diff --git a/src/node_options.cc b/src/node_options.cc
index 49b8eb0bad07d4..af7788c83bc9b1 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -154,8 +154,8 @@ void EnvironmentOptions::CheckOptions(std::vector<std::string>* errors) {
   }
 
   // We may add support for other profiles in the future.
-  if (!prof_dir.empty() && !cpu_prof) {
-    errors->push_back("--prof-dir must be used with --cpu-prof");
+  if (!node_prof_dir.empty() && !cpu_prof) {
+    errors->push_back("--node-prof-dir must be used with --cpu-prof");
   }
 
   debug_options_.CheckOptions(errors);
@@ -351,10 +351,10 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "specified file name of the V8 CPU profile generated with "
             "--cpu-prof",
             &EnvironmentOptions::cpu_prof_name);
-  AddOption("--prof-dir",
+  AddOption("--node-prof-dir",
             "Directory where the V8 profiles generated by --cpu-prof will be "
             "placed. Does not affect --prof.",
-            &EnvironmentOptions::prof_dir);
+            &EnvironmentOptions::node_prof_dir);
 #endif  // HAVE_INSPECTOR
   AddOption("--redirect-warnings",
             "write warnings to file instead of stderr",
diff --git a/src/node_options.h b/src/node_options.h
index 58cc8d502f2492..1749d0b82b8d8e 100644
--- a/src/node_options.h
+++ b/src/node_options.h
@@ -110,7 +110,7 @@ class EnvironmentOptions : public Options {
   bool preserve_symlinks_main = false;
   bool prof_process = false;
 #if HAVE_INSPECTOR
-  std::string prof_dir;
+  std::string node_prof_dir;
   std::string cpu_prof_name;
   bool cpu_prof = false;
 #endif  // HAVE_INSPECTOR
diff --git a/test/sequential/test-cpu-prof.js b/test/sequential/test-cpu-prof.js
index 869982c1f72996..6bdc672cedf190 100644
--- a/test/sequential/test-cpu-prof.js
+++ b/test/sequential/test-cpu-prof.js
@@ -1,6 +1,6 @@
 'use strict';
 
-// This tests that --cpu-prof, --prof-dir and --cpu-prof-name works.
+// This tests that --cpu-prof, --node-prof-dir and --cpu-prof-name works.
 
 const common = require('../common');
 if (process.features.debug &&
@@ -162,11 +162,11 @@ const env = {
     `${process.execPath}: --cpu-prof-name must be used with --cpu-prof`);
 }
 
-// --prof-dir without --cpu-prof
+// --node-prof-dir without --cpu-prof
 {
   tmpdir.refresh();
   const output = spawnSync(process.execPath, [
-    '--prof-dir',
+    '--node-prof-dir',
     'prof',
     fixtures.path('workload', 'fibonacci.js'),
   ], {
@@ -180,7 +180,7 @@ const env = {
   assert.strictEqual(output.status, 9);
   assert.strictEqual(
     stderr,
-    `${process.execPath}: --prof-dir must be used with --cpu-prof`);
+    `${process.execPath}: --node-prof-dir must be used with --cpu-prof`);
 }
 
 // --cpu-prof-name
@@ -205,12 +205,12 @@ const env = {
   verifyFrames(output, file, 'fibonacci.js');
 }
 
-// relative --prof-dir
+// relative --node-prof-dir
 {
   tmpdir.refresh();
   const output = spawnSync(process.execPath, [
     '--cpu-prof',
-    '--prof-dir',
+    '--node-prof-dir',
     'prof',
     fixtures.path('workload', 'fibonacci.js'),
   ], {
@@ -228,13 +228,13 @@ const env = {
   verifyFrames(output, profiles[0], 'fibonacci.js');
 }
 
-// absolute --prof-dir
+// absolute --node-prof-dir
 {
   tmpdir.refresh();
   const dir = path.join(tmpdir.path, 'prof');
   const output = spawnSync(process.execPath, [
     '--cpu-prof',
-    '--prof-dir',
+    '--node-prof-dir',
     dir,
     fixtures.path('workload', 'fibonacci.js'),
   ], {
@@ -251,7 +251,7 @@ const env = {
   verifyFrames(output, profiles[0], 'fibonacci.js');
 }
 
-// --prof-dir and --cpu-prof-name
+// --node-prof-dir and --cpu-prof-name
 {
   tmpdir.refresh();
   const dir = path.join(tmpdir.path, 'prof');
@@ -260,7 +260,7 @@ const env = {
     '--cpu-prof',
     '--cpu-prof-name',
     'test.cpuprofile',
-    '--prof-dir',
+    '--node-prof-dir',
     dir,
     fixtures.path('workload', 'fibonacci.js'),
   ], {
@@ -277,11 +277,11 @@ const env = {
   verifyFrames(output, file, 'fibonacci.js');
 }
 
-// --prof-dir with worker
+// --node-prof-dir with worker
 {
   tmpdir.refresh();
   const output = spawnSync(process.execPath, [
-    '--prof-dir',
+    '--node-prof-dir',
     'prof',
     '--cpu-prof',
     fixtures.path('workload', 'fibonacci-worker.js'),

From 71c8b8922f490f6a1afa5c00443b05b9bcdea3d3 Mon Sep 17 00:00:00 2001
From: Joyee Cheung <joyeec9h3@gmail.com>
Date: Sat, 20 Apr 2019 00:11:45 +0800
Subject: [PATCH 3/4] fixup! squash! inspector: split --cpu-prof-path to
 --prof-dir and --cpu-prof-name

---
 doc/api/cli.md                   | 25 +++++++++++--------------
 doc/node.1                       | 14 ++++++--------
 src/env-inl.h                    |  8 ++++----
 src/env.cc                       |  4 ++--
 src/env.h                        |  6 +++---
 src/inspector_profiler.cc        |  2 +-
 src/node.cc                      |  2 +-
 src/node_options.cc              | 18 +++++++++---------
 src/node_options.h               |  2 +-
 test/sequential/test-cpu-prof.js | 24 ++++++++++++------------
 10 files changed, 50 insertions(+), 55 deletions(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index d01f967ea2897b..150812cfb2e369 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -85,7 +85,7 @@ added: REPLACEME
 Starts the V8 CPU profiler on start up, and writes the CPU profile to disk
 before exit.
 
-If `--node-prof-dir` is not specified, the generated profile will be placed
+If `--cpu-prof-dir` is not specified, the generated profile will be placed
 in the current working directory.
 
 If `--cpu-prof-name` is not specified, the generated profile will be
@@ -97,6 +97,16 @@ $ ls *.cpuprofile
 CPU.20190409.202950.15293.0.0.cpuprofile
 ```
 
+### `--cpu-prof-dir`
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1 - Experimental
+
+Specify the directory where the CPU profiles generated by `--cpu-prof` will
+be placed.
+
 ### `--cpu-prof-name`
 <!-- YAML
 added: REPLACEME
@@ -360,19 +370,6 @@ added: v7.10.0
 
 This option is a no-op. It is kept for compatibility.
 
-### `--node-prof-dir`
-<!-- YAML
-added: REPLACEME
--->
-
-> Stability: 1 - Experimental
-
-Specify the directory where the CPU profiles generated by `--cpu-prof` will
-be placed.
-
-This does not apply to `--prof` whose output is handled by V8 instead of
-Node.js.
-
 ### `--no-deprecation`
 <!-- YAML
 added: v0.8.0
diff --git a/doc/node.1 b/doc/node.1
index 74538002dd4bea..3e8c4fad977353 100644
--- a/doc/node.1
+++ b/doc/node.1
@@ -81,10 +81,15 @@ Print source-able bash completion script for Node.js.
 .It Fl -cpu-prof
 Start the V8 CPU profiler on start up, and write the CPU profile to disk
 before exit. If
-.Fl -node-prof-dir
+.Fl -cpu-prof-dir
 is not specified, the profile will be written to the current working directory
 with a generated file name.
 .
+.It Fl -cpu-prof-dir
+The directory where the CPU profiles generated by
+.Fl -cpu-prof
+will be placed.
+.
 .It Fl -cpu-prof-name
 File name of the V8 CPU profile generated with
 .Fl -cpu-prof
@@ -213,13 +218,6 @@ Specify the maximum size of HTTP headers in bytes. Defaults to 8KB.
 This option is a no-op.
 It is kept for compatibility.
 .
-.It Fl -node-prof-dir
-The directory where the CPU profiles generated by
-.Fl -cpu-prof
-will be placed. Note that this does not apply to
-.Fl -prof
-whose output is handled by V8, not Node.js.
-.
 .It Fl -no-deprecation
 Silence deprecation warnings.
 .
diff --git a/src/env-inl.h b/src/env-inl.h
index feae4c26b0dd05..ee0503620cd66d 100644
--- a/src/env-inl.h
+++ b/src/env-inl.h
@@ -687,12 +687,12 @@ inline const std::string& Environment::cpu_profile_path() const {
   return cpu_profile_path_;
 }
 
-inline void Environment::set_node_prof_dir(const std::string& path) {
-  node_prof_dir_ = path;
+inline void Environment::set_cpu_prof_dir(const std::string& path) {
+  cpu_prof_dir_ = path;
 }
 
-inline const std::string& Environment::node_prof_dir() const {
-  return node_prof_dir_;
+inline const std::string& Environment::cpu_prof_dir() const {
+  return cpu_prof_dir_;
 }
 
 #endif  // HAVE_INSPECTOR
diff --git a/src/env.cc b/src/env.cc
index e705d74de43d11..46df1942f8b04c 100644
--- a/src/env.cc
+++ b/src/env.cc
@@ -849,7 +849,7 @@ void Environment::stop_sub_worker_contexts() {
 
 void Environment::InitializeNodeProfDir(const std::string& dir) {
   if (!dir.empty()) {
-    node_prof_dir_ = dir;
+    cpu_prof_dir_ = dir;
     return;
   }
   char cwd[CWD_BUFSIZE];
@@ -858,7 +858,7 @@ void Environment::InitializeNodeProfDir(const std::string& dir) {
   // TODO(joyeecheung): fallback to exec path / argv[0]
   CHECK_EQ(err, 0);
   CHECK_GT(size, 0);
-  node_prof_dir_ = cwd;
+  cpu_prof_dir_ = cwd;
 }
 
 #endif  // HAVE_INSPECTOR
diff --git a/src/env.h b/src/env.h
index b8606f95fe8060..792b797716ddee 100644
--- a/src/env.h
+++ b/src/env.h
@@ -1138,8 +1138,8 @@ class Environment : public MemoryRetainer {
   inline void set_cpu_profile_path(const std::string& path);
   inline const std::string& cpu_profile_path() const;
 
-  inline void set_node_prof_dir(const std::string& path);
-  inline const std::string& node_prof_dir() const;
+  inline void set_cpu_prof_dir(const std::string& path);
+  inline const std::string& cpu_prof_dir() const;
 
   void InitializeNodeProfDir(const std::string& dir);
 #endif  // HAVE_INSPECTOR
@@ -1178,7 +1178,7 @@ class Environment : public MemoryRetainer {
   std::unique_ptr<profiler::V8CoverageConnection> coverage_connection_;
   std::unique_ptr<profiler::V8CpuProfilerConnection> cpu_profiler_connection_;
   std::string coverage_directory_;
-  std::string node_prof_dir_;
+  std::string cpu_prof_dir_;
   std::string cpu_profile_path_;
 #endif  // HAVE_INSPECTOR
 
diff --git a/src/inspector_profiler.cc b/src/inspector_profiler.cc
index 6fc02b3d8911cc..3507ae4ef2f1e0 100644
--- a/src/inspector_profiler.cc
+++ b/src/inspector_profiler.cc
@@ -319,7 +319,7 @@ void StartCoverageCollection(Environment* env) {
 }
 
 void StartCpuProfiling(Environment* env, const std::string& profile_name) {
-  std::string path = env->node_prof_dir() + std::string(kPathSeparator);
+  std::string path = env->cpu_prof_dir() + std::string(kPathSeparator);
   if (profile_name.empty()) {
     DiagnosticFilename filename(env, "CPU", "cpuprofile");
     path += *filename;
diff --git a/src/node.cc b/src/node.cc
index 8c4e6051262175..472609c778ef8b 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -239,7 +239,7 @@ MaybeLocal<Value> RunBootstrapping(Environment* env) {
 
 #if HAVE_INSPECTOR
   if (env->options()->cpu_prof) {
-    env->InitializeNodeProfDir(env->options()->node_prof_dir);
+    env->InitializeNodeProfDir(env->options()->cpu_prof_dir);
     profiler::StartCpuProfiling(env, env->options()->cpu_prof_name);
   }
 #endif  // HAVE_INSPECTOR
diff --git a/src/node_options.cc b/src/node_options.cc
index af7788c83bc9b1..99f4fda128d1ea 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -149,13 +149,13 @@ void EnvironmentOptions::CheckOptions(std::vector<std::string>* errors) {
   }
 
 #if HAVE_INSPECTOR
-  if (!cpu_prof && !cpu_prof_name.empty()) {
-    errors->push_back("--cpu-prof-name must be used with --cpu-prof");
-  }
-
-  // We may add support for other profiles in the future.
-  if (!node_prof_dir.empty() && !cpu_prof) {
-    errors->push_back("--node-prof-dir must be used with --cpu-prof");
+  if (!cpu_prof) {
+    if (!cpu_prof_name.empty()) {
+      errors->push_back("--cpu-prof-name must be used with --cpu-prof");
+    }
+    if (!cpu_prof_dir.empty()) {
+      errors->push_back("--cpu-prof-dir must be used with --cpu-prof");
+    }
   }
 
   debug_options_.CheckOptions(errors);
@@ -351,10 +351,10 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "specified file name of the V8 CPU profile generated with "
             "--cpu-prof",
             &EnvironmentOptions::cpu_prof_name);
-  AddOption("--node-prof-dir",
+  AddOption("--cpu-prof-dir",
             "Directory where the V8 profiles generated by --cpu-prof will be "
             "placed. Does not affect --prof.",
-            &EnvironmentOptions::node_prof_dir);
+            &EnvironmentOptions::cpu_prof_dir);
 #endif  // HAVE_INSPECTOR
   AddOption("--redirect-warnings",
             "write warnings to file instead of stderr",
diff --git a/src/node_options.h b/src/node_options.h
index 1749d0b82b8d8e..f92261569002e0 100644
--- a/src/node_options.h
+++ b/src/node_options.h
@@ -110,7 +110,7 @@ class EnvironmentOptions : public Options {
   bool preserve_symlinks_main = false;
   bool prof_process = false;
 #if HAVE_INSPECTOR
-  std::string node_prof_dir;
+  std::string cpu_prof_dir;
   std::string cpu_prof_name;
   bool cpu_prof = false;
 #endif  // HAVE_INSPECTOR
diff --git a/test/sequential/test-cpu-prof.js b/test/sequential/test-cpu-prof.js
index 6bdc672cedf190..ffaa0e9b1b8001 100644
--- a/test/sequential/test-cpu-prof.js
+++ b/test/sequential/test-cpu-prof.js
@@ -1,6 +1,6 @@
 'use strict';
 
-// This tests that --cpu-prof, --node-prof-dir and --cpu-prof-name works.
+// This tests that --cpu-prof, --cpu-prof-dir and --cpu-prof-name works.
 
 const common = require('../common');
 if (process.features.debug &&
@@ -162,11 +162,11 @@ const env = {
     `${process.execPath}: --cpu-prof-name must be used with --cpu-prof`);
 }
 
-// --node-prof-dir without --cpu-prof
+// --cpu-prof-dir without --cpu-prof
 {
   tmpdir.refresh();
   const output = spawnSync(process.execPath, [
-    '--node-prof-dir',
+    '--cpu-prof-dir',
     'prof',
     fixtures.path('workload', 'fibonacci.js'),
   ], {
@@ -180,7 +180,7 @@ const env = {
   assert.strictEqual(output.status, 9);
   assert.strictEqual(
     stderr,
-    `${process.execPath}: --node-prof-dir must be used with --cpu-prof`);
+    `${process.execPath}: --cpu-prof-dir must be used with --cpu-prof`);
 }
 
 // --cpu-prof-name
@@ -205,12 +205,12 @@ const env = {
   verifyFrames(output, file, 'fibonacci.js');
 }
 
-// relative --node-prof-dir
+// relative --cpu-prof-dir
 {
   tmpdir.refresh();
   const output = spawnSync(process.execPath, [
     '--cpu-prof',
-    '--node-prof-dir',
+    '--cpu-prof-dir',
     'prof',
     fixtures.path('workload', 'fibonacci.js'),
   ], {
@@ -228,13 +228,13 @@ const env = {
   verifyFrames(output, profiles[0], 'fibonacci.js');
 }
 
-// absolute --node-prof-dir
+// absolute --cpu-prof-dir
 {
   tmpdir.refresh();
   const dir = path.join(tmpdir.path, 'prof');
   const output = spawnSync(process.execPath, [
     '--cpu-prof',
-    '--node-prof-dir',
+    '--cpu-prof-dir',
     dir,
     fixtures.path('workload', 'fibonacci.js'),
   ], {
@@ -251,7 +251,7 @@ const env = {
   verifyFrames(output, profiles[0], 'fibonacci.js');
 }
 
-// --node-prof-dir and --cpu-prof-name
+// --cpu-prof-dir and --cpu-prof-name
 {
   tmpdir.refresh();
   const dir = path.join(tmpdir.path, 'prof');
@@ -260,7 +260,7 @@ const env = {
     '--cpu-prof',
     '--cpu-prof-name',
     'test.cpuprofile',
-    '--node-prof-dir',
+    '--cpu-prof-dir',
     dir,
     fixtures.path('workload', 'fibonacci.js'),
   ], {
@@ -277,11 +277,11 @@ const env = {
   verifyFrames(output, file, 'fibonacci.js');
 }
 
-// --node-prof-dir with worker
+// --cpu-prof-dir with worker
 {
   tmpdir.refresh();
   const output = spawnSync(process.execPath, [
-    '--node-prof-dir',
+    '--cpu-prof-dir',
     'prof',
     '--cpu-prof',
     fixtures.path('workload', 'fibonacci-worker.js'),

From 1d78190a62c3a65b669d0245aad0737bf1995d20 Mon Sep 17 00:00:00 2001
From: Joyee Cheung <joyeec9h3@gmail.com>
Date: Sun, 21 Apr 2019 20:08:58 +0800
Subject: [PATCH 4/4] fixup! fixup! squash! inspector: split --cpu-prof-path to
 --prof-dir and --cpu-prof-name

---
 src/env.cc  | 2 +-
 src/env.h   | 2 +-
 src/node.cc | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/env.cc b/src/env.cc
index 46df1942f8b04c..cbb8ab6fe27c02 100644
--- a/src/env.cc
+++ b/src/env.cc
@@ -847,7 +847,7 @@ void Environment::stop_sub_worker_contexts() {
 
 #if HAVE_INSPECTOR
 
-void Environment::InitializeNodeProfDir(const std::string& dir) {
+void Environment::InitializeCPUProfDir(const std::string& dir) {
   if (!dir.empty()) {
     cpu_prof_dir_ = dir;
     return;
diff --git a/src/env.h b/src/env.h
index 792b797716ddee..def7cba3886ac2 100644
--- a/src/env.h
+++ b/src/env.h
@@ -1141,7 +1141,7 @@ class Environment : public MemoryRetainer {
   inline void set_cpu_prof_dir(const std::string& path);
   inline const std::string& cpu_prof_dir() const;
 
-  void InitializeNodeProfDir(const std::string& dir);
+  void InitializeCPUProfDir(const std::string& dir);
 #endif  // HAVE_INSPECTOR
 
  private:
diff --git a/src/node.cc b/src/node.cc
index 472609c778ef8b..d237ba0d1511e1 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -239,7 +239,7 @@ MaybeLocal<Value> RunBootstrapping(Environment* env) {
 
 #if HAVE_INSPECTOR
   if (env->options()->cpu_prof) {
-    env->InitializeNodeProfDir(env->options()->cpu_prof_dir);
+    env->InitializeCPUProfDir(env->options()->cpu_prof_dir);
     profiler::StartCpuProfiling(env, env->options()->cpu_prof_name);
   }
 #endif  // HAVE_INSPECTOR