Remove V8Call and replace with v8::Maybe<T>::To and v8::MaybeLocal<T>::ToLocal.

third_party/WebKit/Source/bindings/core/v8/ScriptModule.cpp

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "bindings/core/v8/ScriptModule.h"
 
 #include "bindings/core/v8/V8BindingForCore.h"
 #include "core/dom/Modulator.h"
 #include "core/dom/ScriptModuleResolver.h"
 
 namespace blink {
 
 ScriptModule::ScriptModule(v8::Isolate* isolate, v8::Local<v8::Module> module)
     : module_(SharedPersistent<v8::Module>::Create(module, isolate)),
       identity_hash_(static_cast<unsigned>(module->GetIdentityHash())) {
   DCHECK(!module_->IsEmpty());
 }
 
 ScriptModule::~ScriptModule() {}
 
 ScriptModule ScriptModule::Compile(v8::Isolate* isolate,
                                    const String& source,
                                    const String& file_name,
                                    AccessControlStatus access_control_status) {
   v8::TryCatch try_catch(isolate);
   try_catch.SetVerbose(true);
   v8::Local<v8::Module> module;
-  if (!V8Call(V8ScriptRunner::CompileModule(isolate, source, file_name,
-                                            access_control_status),
-              module, try_catch)) {
+  if (!V8ScriptRunner::CompileModule(isolate, source, file_name,
+                                     access_control_status)
+           .ToLocal(&module)) {
     // Compilation error is not used in Blink implementaion logic.
     // Note: Error message is delivered to user (e.g. console) by message
     // listeners set on v8::Isolate. See V8Initializer::initalizeMainThread().
     // TODO(nhiroki): Revisit this when supporting modules on worker threads.
     DCHECK(try_catch.HasCaught());
     return ScriptModule();
   }
   DCHECK(!try_catch.HasCaught());
   return ScriptModule(isolate, module);
 }
@@ skipping 20 matching lines @@
 
   // Isolate exceptions that occur when executing the code. These exceptions
   // should not interfere with javascript code we might evaluate from C++ when
   // returning from here.
   v8::TryCatch try_catch(isolate);
   try_catch.SetVerbose(true);
 
   // TODO(kouhei): We currently don't have a code-path which use return value of
   // EvaluateModule. Stop ignoring result once we have such path.
   v8::Local<v8::Value> result;
-  if (!V8Call(
-          V8ScriptRunner::EvaluateModule(module_->NewLocal(isolate),
-                                         script_state->GetContext(), isolate),
-          result, try_catch)) {
+  if (!V8ScriptRunner::EvaluateModule(module_->NewLocal(isolate),
+                                      script_state->GetContext(), isolate)
+           .ToLocal(&result)) {
     return;
   }
 }
 
 Vector<String> ScriptModule::ModuleRequests(ScriptState* script_state) {
   if (IsNull())
     return Vector<String>();
 
   v8::Local<v8::Module> module = module_->NewLocal(script_state->GetIsolate());
 
@@ skipping 24 matching lines @@
   if (resolved.IsNull()) {
     DCHECK(exception_state.HadException());
     return v8::MaybeLocal<v8::Module>();
   }
 
   DCHECK(!exception_state.HadException());
   return v8::MaybeLocal<v8::Module>(resolved.module_->NewLocal(isolate));
 }
 
 }  // namespace blink