ThreadInfo integration should only be added for non-web platforms (#3144)

Author: denrase

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## Unreleased
+
+### Features
+
+- Tag all spans with thread info on non-web platforms ([#3101](https://github.com/getsentry/sentry-dart/pull/3101), [#3144](https://github.com/getsentry/sentry-dart/pull/3144))
+
 ## 9.7.0-beta.1
 
 ### Features

packages/flutter/lib/src/sentry_flutter.dart

@@ -230,8 +230,9 @@ mixin SentryFlutter {
 
     integrations.add(DebugPrintIntegration());
 
-    integrations.add(ThreadInfoIntegration());
-
+    if (!platform.isWeb) {
+      integrations.add(ThreadInfoIntegration());
+    }
     return integrations;
   }
 

packages/flutter/test/sentry_flutter_test.dart

@@ -12,6 +12,7 @@ import 'package:sentry_flutter/src/integrations/connectivity/connectivity_integr
 import 'package:sentry_flutter/src/integrations/integrations.dart';
 import 'package:sentry_flutter/src/integrations/screenshot_integration.dart';
 import 'package:sentry_flutter/src/integrations/generic_app_start_integration.dart';
+import 'package:sentry_flutter/src/integrations/thread_info_integration.dart';
 import 'package:sentry_flutter/src/integrations/web_session_integration.dart';
 import 'package:sentry_flutter/src/profiling.dart';
 import 'package:sentry_flutter/src/renderer/renderer.dart';
@@ -45,6 +46,7 @@ final linuxWindowsAndWebIntegrations = [
 
 final nonWebIntegrations = [
   OnErrorIntegration,
+  ThreadInfoIntegration,
 ];
 
 // These should be added to Android
@@ -719,14 +721,34 @@ void main() {
                 integration.runtimeType.toString() == 'ThreadInfoIntegration'),
             true,
             reason:
-                'ThreadInfoIntegration should be added when tracing is enabled',
+                'ThreadInfoIntegration should be added on non-web platforms',
           );
         },
         appRunner: appRunner,
         options: sentryFlutterOptions,
       );
       SentryFlutter.native = null;
     });
+
+    test('ThreadInfoIntegration is not added on web', () async {
+      final sentryFlutterOptions =
+          defaultTestOptions(checker: MockRuntimeChecker())
+            ..platform = MockPlatform.linux(isWeb: true)
+            ..methodChannel = native.channel;
+
+      await SentryFlutter.init(
+        (options) {
+          expect(
+            options.integrations.any((integration) =>
+                integration.runtimeType.toString() == 'ThreadInfoIntegration'),
+            false,
+            reason: 'ThreadInfoIntegration should not be added on web platform',
+          );
+        },
+        appRunner: appRunner,
+        options: sentryFlutterOptions,
+      );
+    }, testOn: 'browser');
   });
 
   test('resumeAppHangTracking calls native method when available', () async {