From 7267396aeb02682875dd73e38142c653edac07d2 Mon Sep 17 00:00:00 2001
From: Vadim Gorbachev <bmsdave@gmail.com>
Date: Wed, 6 Nov 2019 19:28:08 +0300
Subject: [PATCH] src: persist strings that are used multiple times in the
 environment

PR-URL: https://github.com/nodejs/node/pull/30321
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
---
 src/heap_utils.cc          | 7 ++++---
 src/node_process_object.cc | 2 +-
 src/stream_wrap.cc         | 2 +-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/heap_utils.cc b/src/heap_utils.cc
index 8391f1de3726c1..b996f78aa4199b 100644
--- a/src/heap_utils.cc
+++ b/src/heap_utils.cc
@@ -89,14 +89,15 @@ class JSGraph : public EmbedderGraph {
   MaybeLocal<Array> CreateObject() const {
     EscapableHandleScope handle_scope(isolate_);
     Local<Context> context = isolate_->GetCurrentContext();
+    Environment* env = Environment::GetCurrent(context);
 
     std::unordered_map<Node*, Local<Object>> info_objects;
     Local<Array> nodes = Array::New(isolate_, nodes_.size());
     Local<String> edges_string = FIXED_ONE_BYTE_STRING(isolate_, "edges");
     Local<String> is_root_string = FIXED_ONE_BYTE_STRING(isolate_, "isRoot");
-    Local<String> name_string = FIXED_ONE_BYTE_STRING(isolate_, "name");
-    Local<String> size_string = FIXED_ONE_BYTE_STRING(isolate_, "size");
-    Local<String> value_string = FIXED_ONE_BYTE_STRING(isolate_, "value");
+    Local<String> name_string = env->name_string();
+    Local<String> size_string = env->size_string();
+    Local<String> value_string = env->value_string();
     Local<String> wraps_string = FIXED_ONE_BYTE_STRING(isolate_, "wraps");
     Local<String> to_string = FIXED_ONE_BYTE_STRING(isolate_, "to");
 
diff --git a/src/node_process_object.cc b/src/node_process_object.cc
index 92124503ab9c5c..a1bf90c8d69fc0 100644
--- a/src/node_process_object.cc
+++ b/src/node_process_object.cc
@@ -74,7 +74,7 @@ MaybeLocal<Object> CreateProcessObject(Environment* env) {
   Local<Context> context = env->context();
 
   Local<FunctionTemplate> process_template = FunctionTemplate::New(isolate);
-  process_template->SetClassName(FIXED_ONE_BYTE_STRING(isolate, "process"));
+  process_template->SetClassName(env->process_string());
   Local<Function> process_ctor;
   Local<Object> process;
   if (!process_template->GetFunction(context).ToLocal(&process_ctor) ||
diff --git a/src/stream_wrap.cc b/src/stream_wrap.cc
index 1ec8386fbbb534..21b775401e4571 100644
--- a/src/stream_wrap.cc
+++ b/src/stream_wrap.cc
@@ -78,7 +78,7 @@ void LibuvStreamWrap::Initialize(Local<Object> target,
   // - callback
   // - handle
   sw->InstanceTemplate()->Set(
-      FIXED_ONE_BYTE_STRING(env->isolate(), "oncomplete"),
+      env->oncomplete_string(),
       v8::Null(env->isolate()));
   sw->InstanceTemplate()->Set(FIXED_ONE_BYTE_STRING(env->isolate(), "callback"),
       v8::Null(env->isolate()));