Fix calls to Object::wrap

Author: gahaas

src/env.cc

@@ -617,7 +617,7 @@ IsolateData::~IsolateData() {}
 // Deprecated API, embedders should use v8::Object::Wrap() directly instead.
 void SetCppgcReference(Isolate* isolate,
                        Local<Object> object,
-                       void* wrappable) {
+                       v8::Object::Wrappable* wrappable) {
   v8::Object::Wrap<v8::CppHeapPointerTag::kDefaultTag>(
       isolate, object, wrappable);
 }

src/node.h

@@ -1567,9 +1567,10 @@ void RegisterSignalHandler(int signal,
 // work with only Node.js versions with v8::Object::Wrap() should use that
 // instead.
 NODE_DEPRECATED("Use v8::Object::Wrap()",
-                NODE_EXTERN void SetCppgcReference(v8::Isolate* isolate,
-                                                   v8::Local<v8::Object> object,
-                                                   void* wrappable));
+                NODE_EXTERN void SetCppgcReference(
+                  v8::Isolate* isolate,
+                  v8::Local<v8::Object> object,
+                  v8::Object::Wrappable* wrappable));
 
 }  // namespace node
 

test/addons/cppgc-object/binding.cc

@@ -8,7 +8,7 @@
 #include <v8.h>
 #include <algorithm>
 
-class CppGCed : public cppgc::GarbageCollected<CppGCed> {
+class CppGCed : public v8::Object::Wrappable {
  public:
   static uint16_t states[2];
   static constexpr int kDestructCount = 0;

test/cctest/test_cppgc.cc

@@ -10,7 +10,7 @@
 // This tests that Node.js can work with an existing CppHeap.
 
 // Mimic a class that does not know about Node.js.
-class CppGCed : public cppgc::GarbageCollected<CppGCed> {
+class CppGCed : public v8::Object::Wrappable {
  public:
   static int kConstructCount;
   static int kDestructCount;