SDKCD: ignore Flutter functions handleDrawFrame and handleBeginFrame (#89472)

SDKCD will falsely report `handleDrawFrame` and `handleBeginFrame` as
SDK crashes although they are not.

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>

Author: 2 people

src/sentry/utils/sdk_crashes/sdk_crash_detection_config.py

@@ -355,10 +355,16 @@ def build_sdk_crash_detection_configs() -> Sequence[SDKCrashDetectionConfig]:
                 },
                 path_replacer=KeepFieldPathReplacer(fields={"package", "filename", "abs_path"}),
             ),
-            # getCurrentStackTrace is always part of the stacktrace when the SDK captures the stacktrace,
-            # and would cause false positives. Therefore, we ignore it.
             sdk_crash_ignore_functions_matchers={
+                # getCurrentStackTrace is always part of the stacktrace when the SDK captures the stacktrace,
+                # and would cause false positives. Therefore, we ignore it.
                 "getCurrentStackTrace",
+                # Ignore handleDrawFrame and handleBeginFrame to avoid false positives.
+                # In the Sentry Flutter SDK, we override the handleDrawFrame and handleBeginFrame methods,
+                # add our custom implementation on top to instrument frame tracking and then forward the calls to Flutter.
+                # However every custom implementation is try/catch guarded so no exception can be thrown.
+                "SentryWidgetsBindingMixin.handleDrawFrame",
+                "SentryWidgetsBindingMixin.handleBeginFrame",
             },
         )
         configs.append(dart_config)

tests/sentry/utils/sdk_crashes/test_sdk_crash_detection_dart.py

@@ -224,3 +224,33 @@ def test_too_low_min_sdk_version_not_detected(
     )
 
     assert mock_sdk_crash_reporter.report.call_count == 0
+
+
+@decorators
+def test_ignore_handle_begin_frame(mock_sdk_crash_reporter, mock_random, store_event, configs):
+    event_data = get_crash_event(sdk_function="SentryWidgetsBindingMixin.handleBeginFrame")
+    event = store_event(data=event_data)
+
+    configs[1].organization_allowlist = [event.project.organization_id]
+
+    sdk_crash_detection.detect_sdk_crash(
+        event=event,
+        configs=configs,
+    )
+
+    assert mock_sdk_crash_reporter.report.call_count == 0
+
+
+@decorators
+def test_ignore_handle_draw_frame(mock_sdk_crash_reporter, mock_random, store_event, configs):
+    event_data = get_crash_event(sdk_function="SentryWidgetsBindingMixin.handleDrawFrame")
+    event = store_event(data=event_data)
+
+    configs[1].organization_allowlist = [event.project.organization_id]
+
+    sdk_crash_detection.detect_sdk_crash(
+        event=event,
+        configs=configs,
+    )
+
+    assert mock_sdk_crash_reporter.report.call_count == 0