Reviewers: jochen,

Description:
[d8] Remove deprecated calls from d8-readline

[email protected]
LOG=n

Please review this at https://codereview.chromium.org/1242973002/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+12, -9 lines):
  M src/d8-readline.cc


Index: src/d8-readline.cc
diff --git a/src/d8-readline.cc b/src/d8-readline.cc
index eca6553af86506162f3c0239ac6539d576e3e3f8..e57b7567e3b5dd4a36701a8e62dcbad371ed5cc2 100644
--- a/src/d8-readline.cc
+++ b/src/d8-readline.cc
@@ -85,7 +85,8 @@ Local<String> ReadLineEditor::Prompt(const char* prompt) {
   result = readline(prompt);
   if (result == NULL) return Local<String>();
   AddHistory(result);
-  return String::NewFromUtf8(isolate_, result);
+  return String::NewFromUtf8(isolate_, result, NewStringType::kNormal)
+      .ToLocalChecked();
 }


@@ -123,21 +124,23 @@ char* ReadLineEditor::CompletionGenerator(const char* text, int state) {
   HandleScope scope(isolate);
   Local<Array> completions;
   if (state == 0) {
-    Local<String> full_text = String::NewFromUtf8(isolate,
-                                                  rl_line_buffer,
-                                                  String::kNormalString,
-                                                  rl_point);
-    completions = Shell::GetCompletions(isolate,
-                                        String::NewFromUtf8(isolate, text),
-                                        full_text);
+    Local<String> full_text =
+ String::NewFromUtf8(isolate, rl_line_buffer, NewStringType::kNormal,
+                            rl_point)
+            .ToLocalChecked();
+    completions = Shell::GetCompletions(
+        isolate, String::NewFromUtf8(isolate, text, NewStringType::kNormal)
+                     .ToLocalChecked(),
+        full_text);
     current_completions.Reset(isolate, completions);
     current_index = 0;
   } else {
     completions = Local<Array>::New(isolate, current_completions);
   }
   if (current_index < completions->Length()) {
+    Local<Context> context(isolate->GetCurrentContext());
     Local<Integer> index = Integer::New(isolate, current_index);
-    Local<Value> str_obj = completions->Get(index);
+ Local<Value> str_obj = completions->Get(context, index).ToLocalChecked();
     current_index++;
     String::Utf8Value str(str_obj);
     return strdup(*str);


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to