Add debounce to capturing screenshots

CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Enhancements
 
 - Cache parsed DSN ([#2365](https://github.com/getsentry/sentry-dart/pull/2365))
+- Add debounce to `ScreenshotWidget` ([#2368](https://github.com/getsentry/sentry-dart/pull/2368))
 
 ## 8.10.0-beta.2
 

flutter/lib/src/event_processor/screenshot_event_processor.dart

@@ -19,6 +19,14 @@ class ScreenshotEventProcessor implements EventProcessor {
   bool get _hasSentryScreenshotWidget =>
       sentryScreenshotWidgetGlobalKey.currentContext != null;
 
+  final _debounceDuration = Duration(milliseconds: 100);
+
+  /// Only apply this event processor every [debounceDuration] duration.
+  DateTime? _lastApplyCall;
+
+  /// The debounce duration for applying this event processor.
+  Duration get debounceDuration => _debounceDuration;
+
   @override
   Future<SentryEvent?> apply(SentryEvent event, Hint hint) async {
     if (event is SentryTransaction) {
@@ -30,6 +38,20 @@ class ScreenshotEventProcessor implements EventProcessor {
         _hasSentryScreenshotWidget) {
       return event;
     }
+
+    // ignore: invalid_use_of_internal_member
+    final now = _options.clock();
+    final difference = _lastApplyCall?.difference(DateTime.now()).abs();
+    _lastApplyCall = now;
+
+    if (difference != null && difference < debounceDuration) {
+      _options.logger(
+        SentryLevel.warning,
+        'Skipping screenshot due to too many calls within a short time frame.',
+      );
+      return event; // Debounce
+    }
+
     final beforeScreenshot = _options.beforeScreenshot;
     if (beforeScreenshot != null) {
       try {

flutter/test/event_processor/screenshot_event_processor_test.dart

@@ -178,6 +178,71 @@ void main() {
       expect(beforeScreenshotHint, hint);
     });
   });
+
+  group("debounce", () {
+    testWidgets("limits added screenshots within debounce timeframe",
+        (tester) async {
+      // Run with real async https://stackoverflow.com/a/54021863
+      await tester.runAsync(() async {
+        final sut = fixture.getSut(FlutterRenderer.canvasKit, false);
+
+        await tester.pumpWidget(SentryScreenshotWidget(
+            child: Text('Catching Pokémon is a snap!',
+                textDirection: TextDirection.ltr)));
+
+        final throwable = Exception();
+
+        final firstEvent = SentryEvent(throwable: throwable);
+        final firstHint = Hint();
+
+        final secondEvent = SentryEvent(throwable: throwable);
+        final secondHint = Hint();
+
+        // ignore: invalid_use_of_internal_member
+        fixture.options.clock = () => DateTime.fromMillisecondsSinceEpoch(0);
+        await sut.apply(firstEvent, firstHint);
+
+        // ignore: invalid_use_of_internal_member
+        fixture.options.clock = () => DateTime.fromMillisecondsSinceEpoch(
+            sut.debounceDuration.inMilliseconds - 1);
+        await sut.apply(secondEvent, secondHint);
+
+        expect(firstHint.screenshot, isNotNull);
+        expect(secondHint.screenshot, isNull);
+      });
+    });
+
+    testWidgets("adds screenshots after debounce timeframe", (tester) async {
+      // Run with real async https://stackoverflow.com/a/54021863
+      await tester.runAsync(() async {
+        final sut = fixture.getSut(FlutterRenderer.canvasKit, false);
+
+        await tester.pumpWidget(SentryScreenshotWidget(
+            child: Text('Catching Pokémon is a snap!',
+                textDirection: TextDirection.ltr)));
+
+        final throwable = Exception();
+
+        final firstEvent = SentryEvent(throwable: throwable);
+        final firstHint = Hint();
+
+        final secondEvent = SentryEvent(throwable: throwable);
+        final secondHint = Hint();
+
+        // ignore: invalid_use_of_internal_member
+        fixture.options.clock = () => DateTime.fromMillisecondsSinceEpoch(0);
+        await sut.apply(firstEvent, firstHint);
+
+        // ignore: invalid_use_of_internal_member
+        fixture.options.clock = () => DateTime.fromMillisecondsSinceEpoch(
+            sut.debounceDuration.inMilliseconds);
+        await sut.apply(secondEvent, secondHint);
+
+        expect(firstHint.screenshot, isNotNull);
+        expect(secondHint.screenshot, isNotNull);
+      });
+    });
+  });
 }
 
 class Fixture {