src: remove MarkIndependent() calls

The method has been deprecated in upstream V8, with messaging indicating that it is the default for handles to be independent now anyway. PR-URL: #20108 Refs: v8/v8@71ad48f Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Matteo Collina <[email protected]>
nodejs · Apr 22, 2018 · 1f3fbd4 · 1f3fbd4
1 parent aaea706
commit 1f3fbd4
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 6 deletions.
diff --git a/src/base_object-inl.h b/src/base_object-inl.h
@@ -70,7 +70,6 @@ inline void BaseObject::MakeWeak(Type* ptr) {
   v8::Local<v8::Object> handle = object();
   CHECK_GT(handle->InternalFieldCount(), 0);
   Wrap(handle, ptr);
-  persistent_handle_.MarkIndependent();
   persistent_handle_.SetWeak<Type>(ptr, WeakCallback<Type>,
                                    v8::WeakCallbackType::kParameter);
 }

diff --git a/src/node_api.cc b/src/node_api.cc
@@ -387,7 +387,6 @@ class Reference : private Finalizer {
     if (initial_refcount == 0) {
       _persistent.SetWeak(
           this, FinalizeCallback, v8::WeakCallbackType::kParameter);
-      _persistent.MarkIndependent();
     }
   }
 
@@ -431,7 +430,6 @@ class Reference : private Finalizer {
     if (--_refcount == 0) {
       _persistent.SetWeak(
           this, FinalizeCallback, v8::WeakCallbackType::kParameter);
-      _persistent.MarkIndependent();
     }
 
     return _refcount;

diff --git a/src/node_buffer.cc b/src/node_buffer.cc
@@ -144,7 +144,6 @@ CallbackInfo::CallbackInfo(Isolate* isolate,
 
   persistent_.SetWeak(this, WeakCallback, v8::WeakCallbackType::kParameter);
   persistent_.SetWrapperClassId(BUFFER_ID);
-  persistent_.MarkIndependent();
   isolate->AdjustAmountOfExternalAllocatedMemory(sizeof(*this));
 }
 

diff --git a/src/node_contextify.cc b/src/node_contextify.cc
@@ -104,7 +104,6 @@ ContextifyContext::ContextifyContext(
   if (context_.IsEmpty())
     return;
   context_.SetWeak(this, WeakCallback, v8::WeakCallbackType::kParameter);
-  context_.MarkIndependent();
 }
 
 

diff --git a/src/node_object_wrap.h b/src/node_object_wrap.h
@@ -83,7 +83,6 @@ class ObjectWrap {
 
   inline void MakeWeak(void) {
     persistent().SetWeak(this, WeakCallback, v8::WeakCallbackType::kParameter);
-    persistent().MarkIndependent();
   }
 
   /* Ref() marks the object as being attached to an event loop.