update

Author: buenaflor

dart/lib/src/sentry_tracer.dart

@@ -109,15 +109,14 @@ class SentryTracer extends ISentrySpan {
       }
 
       var _rootEndTimestamp = commonEndTimestamp;
-
       if (_trimEnd && children.isNotEmpty) {
         final childEndTimestamps = children
             .where((child) => child.endTimestamp != null)
             .map((child) => child.endTimestamp!);
 
         if (childEndTimestamps.isNotEmpty) {
           final oldestChildEndTimestamp =
-          childEndTimestamps.reduce((a, b) => a.isAfter(b) ? a : b);
+              childEndTimestamps.reduce((a, b) => a.isAfter(b) ? a : b);
           if (_rootEndTimestamp.isAfter(oldestChildEndTimestamp)) {
             _rootEndTimestamp = oldestChildEndTimestamp;
           }
@@ -363,7 +362,8 @@ class SentryTracer extends ISentrySpan {
       Dsn.parse(_hub.options.dsn!).publicKey,
       release: _hub.options.release,
       environment: _hub.options.environment,
-      userId: null, // because of PII not sending it for now
+      userId: null,
+      // because of PII not sending it for now
       userSegment: user?.segment,
       transaction:
           _isHighQualityTransactionName(transactionNameSource) ? name : null,

flutter/lib/src/navigation/sentry_navigator_observer.dart

@@ -212,16 +212,16 @@ class SentryNavigatorObserver extends RouteObserver<PageRoute<dynamic>> {
     }
   }
 
-  Future<ISentrySpan?> _startTransaction(Route<dynamic>? route) async {
+  Future<void> _startTransaction(Route<dynamic>? route) async {
     if (!_enableAutoTransactions) {
-      return null;
+      return;
     }
 
     String? name = _getRouteName(route);
     final arguments = route?.settings.arguments;
 
     if (name == null) {
-      return null;
+      return;
     }
 
     if (name == '/') {
@@ -260,7 +260,7 @@ class SentryNavigatorObserver extends RouteObserver<PageRoute<dynamic>> {
     // if _enableAutoTransactions is enabled but there's no traces sample rate
     if (_transaction is NoOpSentrySpan) {
       _transaction = null;
-      return null;
+      return;
     }
 
     if (arguments != null) {
@@ -272,18 +272,15 @@ class SentryNavigatorObserver extends RouteObserver<PageRoute<dynamic>> {
     });
 
     await _native?.beginNativeFramesCollection();
-
-    return _transaction;
   }
 
   Future<void> _startTimeToDisplayTracking(Route<dynamic>? route) async {
     _completedDisplayTracking = Completer<void>();
-    final routeName = _getRouteName(route);
-    _currentRouteName = routeName;
-
     final arguments = route?.settings.arguments;
-    final transaction = await _startTransaction(route);
+    await _startTransaction(route);
+    final transaction = _transaction;
     if (transaction == null) return;
+    final routeName = _getRouteName(route);
     await _timeToDisplayTracker?.startTracking(
         transaction, routeName, arguments);
     _completedDisplayTracking?.complete();