commit e753844cd4ede3c9ff37d4c96fa1fc978027efe4
author Yegor <yjbanov@google.com> Tue Apr 18 16:49:00 2023 -0700
committer GitHub <noreply@github.com> Tue Apr 18 16:49:00 2023 -0700
tree 21d3b45bdaedc8dfd16a8a9f2d78433700e3bcc7
parent c472346deab450d6e3cd73a66c2803b26f6eea70

[web:canvaskit] remove unnecessary instrumentation from picture (#41313)

Remove unnecessary instrumentation from picture. `UniqueRef`
automatically instruments all allocations, disposals, and garbage
collections, which covers `CkPicture`.

lib/web_ui/lib/src/engine/canvaskit/picture.dart

diff --git a/lib/web_ui/lib/src/engine/canvaskit/picture.dart b/lib/web_ui/lib/src/engine/canvaskit/picture.dart
index 5fdc0cc..19e1d71 100644
--- a/lib/web_ui/lib/src/engine/canvaskit/picture.dart
+++ b/lib/web_ui/lib/src/engine/canvaskit/picture.dart
@@ -6,7 +6,6 @@
 
 import 'package:ui/ui.dart' as ui;
 
-import '../profiler.dart';
 import '../util.dart';
 import 'canvas.dart';
 import 'canvaskit_api.dart';
@@ -77,9 +76,6 @@
       return true;
     }());
     ui.Picture.onDispose?.call(this);
-    if (Instrumentation.enabled) {
-      Instrumentation.instance.incrementCounter('Picture disposed');
-    }
     _isDisposed = true;
     _ref.dispose();
   }

lib/web_ui/test/canvaskit/native_memory_test.dart

diff --git a/lib/web_ui/test/canvaskit/native_memory_test.dart b/lib/web_ui/test/canvaskit/native_memory_test.dart
index 42a99d2..3ccaa13 100644
--- a/lib/web_ui/test/canvaskit/native_memory_test.dart
+++ b/lib/web_ui/test/canvaskit/native_memory_test.dart
@@ -81,6 +81,45 @@
       // there must not be anything else calling into this object for anything
       // useful.
     });
+
+    test('dispose instrumentation', () {
+      Instrumentation.enabled = true;
+      Instrumentation.instance.debugCounters.clear();
+
+      final Object owner = Object();
+      final TestSkDeletable nativeObject = TestSkDeletable();
+
+      expect(Instrumentation.instance.debugCounters, <String, int>{});
+      final UniqueRef<TestSkDeletable> ref = UniqueRef<TestSkDeletable>(owner, nativeObject, 'TestSkDeletable');
+      expect(Instrumentation.instance.debugCounters, <String, int>{
+        'TestSkDeletable Created': 1,
+      });
+      ref.dispose();
+      expect(Instrumentation.instance.debugCounters, <String, int>{
+        'TestSkDeletable Created': 1,
+        'TestSkDeletable Deleted': 1,
+      });
+    });
+
+    test('collect instrumentation', () {
+      Instrumentation.enabled = true;
+      Instrumentation.instance.debugCounters.clear();
+
+      final Object owner = Object();
+      final TestSkDeletable nativeObject = TestSkDeletable();
+
+      expect(Instrumentation.instance.debugCounters, <String, int>{});
+      final UniqueRef<TestSkDeletable> ref = UniqueRef<TestSkDeletable>(owner, nativeObject, 'TestSkDeletable');
+      expect(Instrumentation.instance.debugCounters, <String, int>{
+        'TestSkDeletable Created': 1,
+      });
+      ref.collect();
+      expect(Instrumentation.instance.debugCounters, <String, int>{
+        'TestSkDeletable Created': 1,
+        'TestSkDeletable Leaked': 1,
+        'TestSkDeletable Deleted': 1,
+      });
+    });
   });
 
   group(CountedRef, () {