From cae9ff256be348101ff3655c4a420945970baa19 Mon Sep 17 00:00:00 2001
From: Sarat Addepalli <sarat.addepalli@paytm.com>
Date: Tue, 20 Mar 2018 23:29:17 +0530
Subject: [PATCH] src: fix upcoming V8 deprecation warnings
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

PR-URL: https://github.com/nodejs/node/pull/19490
Fixes: https://github.com/nodejs/node/issues/18909
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
---
 src/inspector_js_api.cc |  2 +-
 src/node.cc             | 12 +++++++-----
 src/node_api.cc         |  2 +-
 src/node_buffer.cc      |  4 ++--
 src/node_contextify.cc  | 12 ++++++------
 src/node_crypto.cc      |  2 +-
 src/node_v8.cc          |  2 +-
 src/string_bytes.cc     |  6 +++---
 8 files changed, 22 insertions(+), 20 deletions(-)

diff --git a/src/inspector_js_api.cc b/src/inspector_js_api.cc
index 1cced9420aea6c..e727410202b119 100644
--- a/src/inspector_js_api.cc
+++ b/src/inspector_js_api.cc
@@ -232,7 +232,7 @@ static void AsyncTaskScheduledWrapper(const FunctionCallbackInfo<Value>& args) {
 
   CHECK(args[0]->IsString());
   Local<String> task_name = args[0].As<String>();
-  String::Value task_name_value(task_name);
+  String::Value task_name_value(args.GetIsolate(), task_name);
   StringView task_name_view(*task_name_value, task_name_value.length());
 
   CHECK(args[1]->IsNumber());
diff --git a/src/node.cc b/src/node.cc
index 35a1d5cb598d5b..a180f97cd4140e 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -1595,9 +1595,11 @@ void AppendExceptionLine(Environment* env,
   ScriptOrigin origin = message->GetScriptOrigin();
   node::Utf8Value filename(env->isolate(), message->GetScriptResourceName());
   const char* filename_string = *filename;
-  int linenum = message->GetLineNumber();
+  int linenum = message->GetLineNumber(env->context()).FromJust();
   // Print line of source code.
-  node::Utf8Value sourceline(env->isolate(), message->GetSourceLine());
+  MaybeLocal<String> source_line_maybe = message->GetSourceLine(env->context());
+  node::Utf8Value sourceline(env->isolate(),
+                             source_line_maybe.ToLocalChecked());
   const char* sourceline_string = *sourceline;
   if (strstr(sourceline_string, "node-do-not-add-exception-line") != nullptr)
     return;
@@ -1745,7 +1747,7 @@ static void ReportException(Environment* env,
         name.IsEmpty() ||
         name->IsUndefined()) {
       // Not an error object. Just print as-is.
-      String::Utf8Value message(er);
+      String::Utf8Value message(env->isolate(), er);
 
       PrintErrorString("%s\n", *message ? *message :
                                           "<toString() threw exception>");
@@ -1797,13 +1799,13 @@ static Local<Value> ExecuteString(Environment* env,
     exit(3);
   }
 
-  Local<Value> result = script.ToLocalChecked()->Run();
+  MaybeLocal<Value> result = script.ToLocalChecked()->Run(env->context());
   if (result.IsEmpty()) {
     ReportException(env, try_catch);
     exit(4);
   }
 
-  return scope.Escape(result);
+  return scope.Escape(result.ToLocalChecked());
 }
 
 
diff --git a/src/node_api.cc b/src/node_api.cc
index ea6bccef3f77de..d1e99495b45ddb 100644
--- a/src/node_api.cc
+++ b/src/node_api.cc
@@ -3325,7 +3325,7 @@ class Work : public node::AsyncResource {
                 void* data = nullptr)
     : AsyncResource(env->isolate,
                     async_resource,
-                    *v8::String::Utf8Value(async_resource_name)),
+                    *v8::String::Utf8Value(env->isolate, async_resource_name)),
     _env(env),
     _data(data),
     _execute(execute),
diff --git a/src/node_buffer.cc b/src/node_buffer.cc
index f9a807602f612c..773b3341821f82 100644
--- a/src/node_buffer.cc
+++ b/src/node_buffer.cc
@@ -946,7 +946,7 @@ void IndexOfString(const FunctionCallbackInfo<Value>& args) {
   size_t result = haystack_length;
 
   if (enc == UCS2) {
-    String::Value needle_value(needle);
+    String::Value needle_value(args.GetIsolate(), needle);
     if (*needle_value == nullptr)
       return args.GetReturnValue().Set(-1);
 
@@ -979,7 +979,7 @@ void IndexOfString(const FunctionCallbackInfo<Value>& args) {
     }
     result *= 2;
   } else if (enc == UTF8) {
-    String::Utf8Value needle_value(needle);
+    String::Utf8Value needle_value(args.GetIsolate(), needle);
     if (*needle_value == nullptr)
       return args.GetReturnValue().Set(-1);
 
diff --git a/src/node_contextify.cc b/src/node_contextify.cc
index f49a2362769bc2..e7f956e9882689 100644
--- a/src/node_contextify.cc
+++ b/src/node_contextify.cc
@@ -1098,21 +1098,21 @@ class ContextifyScript : public BaseObject {
         PersistentToLocal(env->isolate(), wrapped_script->script_);
     Local<Script> script = unbound_script->BindToCurrentContext();
 
-    Local<Value> result;
+    MaybeLocal<Value> result;
     bool timed_out = false;
     bool received_signal = false;
     if (break_on_sigint && timeout != -1) {
       Watchdog wd(env->isolate(), timeout, &timed_out);
       SigintWatchdog swd(env->isolate(), &received_signal);
-      result = script->Run();
+      result = script->Run(env->context());
     } else if (break_on_sigint) {
       SigintWatchdog swd(env->isolate(), &received_signal);
-      result = script->Run();
+      result = script->Run(env->context());
     } else if (timeout != -1) {
       Watchdog wd(env->isolate(), timeout, &timed_out);
-      result = script->Run();
+      result = script->Run(env->context());
     } else {
-      result = script->Run();
+      result = script->Run(env->context());
     }
 
     if (timed_out || received_signal) {
@@ -1143,7 +1143,7 @@ class ContextifyScript : public BaseObject {
       return false;
     }
 
-    args.GetReturnValue().Set(result);
+    args.GetReturnValue().Set(result.ToLocalChecked());
     return true;
   }
 
diff --git a/src/node_crypto.cc b/src/node_crypto.cc
index a3a54b28bd9113..89e2fdc6d6ab0b 100644
--- a/src/node_crypto.cc
+++ b/src/node_crypto.cc
@@ -4807,7 +4807,7 @@ void PublicKeyCipher::Cipher(const FunctionCallbackInfo<Value>& args) {
 
   int padding = args[2]->Uint32Value();
 
-  String::Utf8Value passphrase(args[3]);
+  String::Utf8Value passphrase(args.GetIsolate(), args[3]);
 
   unsigned char* out_value = nullptr;
   size_t out_len = 0;
diff --git a/src/node_v8.cc b/src/node_v8.cc
index e2ffedcea4b1b3..d1514cb0973dbe 100644
--- a/src/node_v8.cc
+++ b/src/node_v8.cc
@@ -114,7 +114,7 @@ void UpdateHeapSpaceStatisticsBuffer(const FunctionCallbackInfo<Value>& args) {
 
 void SetFlagsFromString(const FunctionCallbackInfo<Value>& args) {
   CHECK(args[0]->IsString());
-  String::Utf8Value flags(args[0]);
+  String::Utf8Value flags(args.GetIsolate(), args[0]);
   V8::SetFlagsFromString(*flags, flags.length());
 }
 
diff --git a/src/string_bytes.cc b/src/string_bytes.cc
index 60e88c45888944..8fba3f9921c6cd 100644
--- a/src/string_bytes.cc
+++ b/src/string_bytes.cc
@@ -406,7 +406,7 @@ size_t StringBytes::Write(Isolate* isolate,
       if (is_extern) {
         nbytes = base64_decode(buf, buflen, data, external_nbytes);
       } else {
-        String::Value value(str);
+        String::Value value(isolate, str);
         nbytes = base64_decode(buf, buflen, *value, value.length());
       }
       if (chars_written != nullptr) {
@@ -418,7 +418,7 @@ size_t StringBytes::Write(Isolate* isolate,
       if (is_extern) {
         nbytes = hex_decode(buf, buflen, data, external_nbytes);
       } else {
-        String::Value value(str);
+        String::Value value(isolate, str);
         nbytes = hex_decode(buf, buflen, *value, value.length());
       }
       if (chars_written != nullptr) {
@@ -528,7 +528,7 @@ size_t StringBytes::Size(Isolate* isolate,
       break;
 
     case BASE64: {
-      String::Value value(str);
+      String::Value value(isolate, str);
       data_size = base64_decoded_size(*value, value.length());
       break;
     }