Fix issues

Author: bdero

lib/gpu/gpu.cc

@@ -15,14 +15,20 @@
 
 namespace flutter {
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 IMPLEMENT_WRAPPERTYPEINFO(gpu, FlutterGpuTestClass);
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
+FlutterGpuTestClass::~FlutterGpuTestClass() = default;
+
 }  // namespace flutter
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 uint32_t FlutterGpuTestProc() {
   return 1;
 }
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 Dart_Handle FlutterGpuTestProcWithCallback(Dart_Handle callback) {
   flutter::UIDartState::ThrowIfUIOperationsProhibited();
   if (!Dart_IsClosure(callback)) {
@@ -34,13 +40,21 @@ Dart_Handle FlutterGpuTestProcWithCallback(Dart_Handle callback) {
   return Dart_Null();
 }
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 void FlutterGpuTestClass_Create(Dart_Handle wrapper) {
   auto res = fml::MakeRefCounted<flutter::FlutterGpuTestClass>();
   res->AssociateWithDartWrapper(wrapper);
   FML_LOG(ERROR) << "FlutterGpuTestClass Wrapped.";
 }
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 void FlutterGpuTestClass_Method(flutter::FlutterGpuTestClass* self,
                                 int something) {
   FML_LOG(ERROR) << "Something: " << something;
 }
+
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
+void FlutterGpuTestClass_Dispose(flutter::FlutterGpuTestClass* self,
+                                 int something) {
+  self->ClearDartWrapper();
+}

lib/gpu/gpu.dart

@@ -31,8 +31,11 @@ base class FlutterGpuTestClass extends NativeFieldWrapperClass1 {
   external void _constructor();
 
   /// This is a method that will supply a pointer to the C data counterpart when
-  /// calling the function
+  /// calling the gunction
   @Native<Void Function(Pointer<Void>, Int)>(
       symbol: 'FlutterGpuTestClass_Method')
   external void coolMethod(int something);
+
+  @Native<Void Function(Pointer<Void>)>(symbol: 'FlutterGpuTestClass_Dispose')
+  external void dispose();
 }

lib/gpu/gpu.h

@@ -20,26 +20,33 @@
 
 namespace flutter {
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 class FlutterGpuTestClass
     : public RefCountedDartWrappable<FlutterGpuTestClass> {
   DEFINE_WRAPPERTYPEINFO();
+  FML_FRIEND_MAKE_REF_COUNTED(FlutterGpuTestClass);
 
  public:
+  ~FlutterGpuTestClass() override;
 };
 
 }  // namespace flutter
 
 extern "C" {
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 FLUTTER_EXPORT
 extern uint32_t FlutterGpuTestProc();
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 FLUTTER_EXPORT
 extern Dart_Handle FlutterGpuTestProcWithCallback(Dart_Handle callback);
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 FLUTTER_EXPORT
 extern void FlutterGpuTestClass_Create(Dart_Handle wrapper);
 
+// TODO(131346): Remove this once we migrate the Dart GPU API into this space.
 FLUTTER_EXPORT
 extern void FlutterGpuTestClass_Method(flutter::FlutterGpuTestClass* self,
                                        int something);

testing/dart/gpu_test.dart

@@ -10,10 +10,16 @@ import 'package:litetest/litetest.dart';
 import '../../lib/gpu/gpu.dart' as gpu;
 
 void main() {
+  // TODO(131346): Remove this once we migrate the Dart GPU API into this space.
   test('no crash', () async {
     final int result = gpu.testProc();
     expect(result, 1);
 
+    final String? message = gpu.testProcWithCallback((int result) {
+      expect(result, 1234);
+    });
+    expect(message, null);
+
     final gpu.FlutterGpuTestClass a = gpu.FlutterGpuTestClass();
     a.coolMethod(9847);
   });