diff --git a/src/async_wrap.cc b/src/async_wrap.cc index b7d59671908564..42fd723e4776c0 100644 --- a/src/async_wrap.cc +++ b/src/async_wrap.cc @@ -28,6 +28,8 @@ #include "v8-profiler.h" using v8::Context; +using v8::DontDelete; +using v8::EscapableHandleScope; using v8::Function; using v8::FunctionCallbackInfo; using v8::FunctionTemplate; @@ -36,16 +38,22 @@ using v8::Integer; using v8::Isolate; using v8::Local; using v8::MaybeLocal; +using v8::NewStringType; using v8::Number; using v8::Object; using v8::ObjectTemplate; using v8::Promise; using v8::PromiseHookType; +using v8::PropertyAttribute; using v8::PropertyCallbackInfo; +using v8::ReadOnly; using v8::String; +using v8::TryCatch; using v8::Uint32; using v8::Undefined; using v8::Value; +using v8::WeakCallbackInfo; +using v8::WeakCallbackType; using AsyncHooks = node::Environment::AsyncHooks; @@ -115,7 +123,7 @@ void Emit(Environment* env, double async_id, AsyncHooks::Fields type, if (async_hooks->fields()[type] == 0 || !env->can_call_into_js()) return; - v8::HandleScope handle_scope(env->isolate()); + HandleScope handle_scope(env->isolate()); Local async_id_value = Number::New(env->isolate(), async_id); FatalTryCatch try_catch(env); USE(fn->Call(env->context(), Undefined(env->isolate()), 1, &async_id_value)); @@ -351,8 +359,7 @@ class DestroyParam { Persistent propBag; }; - -void AsyncWrap::WeakCallback(const v8::WeakCallbackInfo& info) { +void AsyncWrap::WeakCallback(const WeakCallbackInfo& info) { HandleScope scope(info.GetIsolate()); std::unique_ptr p{info.GetParameter()}; @@ -377,8 +384,7 @@ static void RegisterDestroyHook(const FunctionCallbackInfo& args) { p->env = Environment::GetCurrent(args); p->target.Reset(isolate, args[0].As()); p->propBag.Reset(isolate, args[2].As()); - p->target.SetWeak( - p, AsyncWrap::WeakCallback, v8::WeakCallbackType::kParameter); + p->target.SetWeak(p, AsyncWrap::WeakCallback, WeakCallbackType::kParameter); } @@ -452,8 +458,8 @@ void AsyncWrap::Initialize(Local target, env->SetMethod(target, "disablePromiseHook", DisablePromiseHook); env->SetMethod(target, "registerDestroyHook", RegisterDestroyHook); - v8::PropertyAttribute ReadOnlyDontDelete = - static_cast(v8::ReadOnly | v8::DontDelete); + PropertyAttribute ReadOnlyDontDelete = + static_cast(ReadOnly | DontDelete); #define FORCE_SET_TARGET_FIELD(obj, str, field) \ (obj)->DefineOwnProperty(context, \ @@ -695,7 +701,7 @@ MaybeLocal AsyncWrap::MakeCallback(const Local cb, async_id AsyncHooksGetExecutionAsyncId(Isolate* isolate) { // Environment::GetCurrent() allocates a Local<> handle. - v8::HandleScope handle_scope(isolate); + HandleScope handle_scope(isolate); Environment* env = Environment::GetCurrent(isolate); if (env == nullptr) return -1; return env->execution_async_id(); @@ -704,7 +710,7 @@ async_id AsyncHooksGetExecutionAsyncId(Isolate* isolate) { async_id AsyncHooksGetTriggerAsyncId(Isolate* isolate) { // Environment::GetCurrent() allocates a Local<> handle. - v8::HandleScope handle_scope(isolate); + HandleScope handle_scope(isolate); Environment* env = Environment::GetCurrent(isolate); if (env == nullptr) return -1; return env->trigger_async_id(); @@ -715,18 +721,18 @@ async_context EmitAsyncInit(Isolate* isolate, Local resource, const char* name, async_id trigger_async_id) { - v8::HandleScope handle_scope(isolate); + HandleScope handle_scope(isolate); Local type = - String::NewFromUtf8(isolate, name, v8::NewStringType::kInternalized) + String::NewFromUtf8(isolate, name, NewStringType::kInternalized) .ToLocalChecked(); return EmitAsyncInit(isolate, resource, type, trigger_async_id); } async_context EmitAsyncInit(Isolate* isolate, Local resource, - v8::Local name, + Local name, async_id trigger_async_id) { - v8::HandleScope handle_scope(isolate); + HandleScope handle_scope(isolate); Environment* env = Environment::GetCurrent(isolate); CHECK_NOT_NULL(env); @@ -748,7 +754,7 @@ async_context EmitAsyncInit(Isolate* isolate, void EmitAsyncDestroy(Isolate* isolate, async_context asyncContext) { // Environment::GetCurrent() allocates a Local<> handle. - v8::HandleScope handle_scope(isolate); + HandleScope handle_scope(isolate); AsyncWrap::EmitDestroy( Environment::GetCurrent(isolate), asyncContext.async_id); } @@ -767,10 +773,10 @@ Local AsyncWrap::GetOwner() { } Local AsyncWrap::GetOwner(Environment* env, Local obj) { - v8::EscapableHandleScope handle_scope(env->isolate()); + EscapableHandleScope handle_scope(env->isolate()); CHECK(!obj.IsEmpty()); - v8::TryCatch ignore_exceptions(env->isolate()); + TryCatch ignore_exceptions(env->isolate()); while (true) { Local owner; if (!obj->Get(env->context(),