Upgrade to the latest Flutter SDK

.github/workflows/build.yaml

@@ -74,7 +74,7 @@ jobs:
           wget -qO- https://dcm.dev/pgp-key.public | sudo gpg --dearmor -o /usr/share/keyrings/dcm.gpg
           echo 'deb [signed-by=/usr/share/keyrings/dcm.gpg arch=amd64] https://dcm.dev/debian stable main' | sudo tee /etc/apt/sources.list.d/dart_stable.list
           sudo apt-get update
-          sudo apt-get install dcm=1.28.0-1 # To avoid errors add `-1` (build number) to the version
+          sudo apt-get install dcm=1.29.0-1 # To avoid errors add `-1` (build number) to the version
           sudo chmod +x /usr/bin/dcm
           echo "$(dcm --version)"
       - name: Setup Dart SDK

flutter-candidate.txt

@@ -1 +1 @@
-36ea2bdeab611e908967b6fa57659998f600a2cb
+a534b2138030409fb79079540d772602dcb7e7df

packages/devtools_app/benchmark/web_bundle_size_test.dart

@@ -11,7 +11,7 @@ import 'package:path/path.dart' as path;
 import 'package:test/test.dart';
 
 // Benchmark size in kB.
-const bundleSizeBenchmark = 5400;
+const bundleSizeBenchmark = 5500;
 const gzipBundleSizeBenchmark = 1650;
 
 void main() {

packages/devtools_app/lib/src/framework/notifications_view.dart

@@ -208,10 +208,10 @@ class _NotificationState extends State<_Notification>
             widget.remove(widget);
           }
         });
-        controller.reverse();
+        unawaited(controller.reverse());
       });
     }
-    controller.forward();
+    unawaited(controller.forward());
   }
 
   @override

packages/devtools_app/lib/src/screens/inspector/layout_explorer/ui/layout_explorer_widget.dart

@@ -167,12 +167,12 @@ abstract class LayoutExplorerWidgetState<
 
   void _animateProperties() {
     if (_animatedProperties != null) {
-      changeController.forward();
+      unawaited(changeController.forward());
     }
     if (_previousProperties != null) {
-      entranceController.reverse();
+      unawaited(entranceController.reverse());
     } else {
-      entranceController.forward();
+      unawaited(entranceController.forward());
     }
   }
 
@@ -214,7 +214,7 @@ abstract class LayoutExplorerWidgetState<
     updateHighlighted(nextProperties);
     setState(() {
       _animatedProperties = computeAnimatedProperties(nextProperties);
-      changeController.forward(from: 0.0);
+      unawaited(changeController.forward(from: 0.0));
     });
   }
 
@@ -237,7 +237,7 @@ abstract class LayoutExplorerWidgetState<
         if (status == AnimationStatus.dismissed) {
           setState(() {
             _previousProperties = null;
-            entranceController.forward();
+            unawaited(entranceController.forward());
           });
         }
       });

packages/devtools_app/lib/src/screens/inspector_shared/inspector_settings_dialog.dart

@@ -130,25 +130,25 @@ class InspectorDefaultDetailsViewOption extends StatelessWidget {
               style: theme.subtleTextStyle,
             ),
             const SizedBox(height: denseSpacing),
-            Row(
-              mainAxisSize: MainAxisSize.min,
-              children: [
-                Radio<InspectorDetailsViewType>(
-                  value: InspectorDetailsViewType.layoutExplorer,
-                  materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
-                  groupValue: selection,
-                  onChanged: _onChanged,
-                ),
-                Text(InspectorDetailsViewType.layoutExplorer.key),
-                const SizedBox(width: denseSpacing),
-                Radio<InspectorDetailsViewType>(
-                  materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
-                  value: InspectorDetailsViewType.widgetDetailsTree,
-                  groupValue: selection,
-                  onChanged: _onChanged,
-                ),
-                Text(InspectorDetailsViewType.widgetDetailsTree.key),
-              ],
+            RadioGroup(
+              groupValue: selection,
+              onChanged: _onChanged,
+              child: Row(
+                mainAxisSize: MainAxisSize.min,
+                children: [
+                  const Radio<InspectorDetailsViewType>(
+                    value: InspectorDetailsViewType.layoutExplorer,
+                    materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
+                  ),
+                  Text(InspectorDetailsViewType.layoutExplorer.key),
+                  const SizedBox(width: denseSpacing),
+                  const Radio<InspectorDetailsViewType>(
+                    materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
+                    value: InspectorDetailsViewType.widgetDetailsTree,
+                  ),
+                  Text(InspectorDetailsViewType.widgetDetailsTree.key),
+                ],
+              ),
             ),
           ],
         );

packages/devtools_app/lib/src/screens/inspector_v2/layout_explorer/ui/layout_explorer_widget.dart

@@ -167,12 +167,12 @@ abstract class LayoutExplorerWidgetState<
 
   void _animateProperties() {
     if (_animatedProperties != null) {
-      changeController.forward();
+      unawaited(changeController.forward());
     }
     if (_previousProperties != null) {
-      entranceController.reverse();
+      unawaited(entranceController.reverse());
     } else {
-      entranceController.forward();
+      unawaited(entranceController.forward());
     }
   }
 
@@ -214,7 +214,7 @@ abstract class LayoutExplorerWidgetState<
     updateHighlighted(nextProperties);
     setState(() {
       _animatedProperties = computeAnimatedProperties(nextProperties);
-      changeController.forward(from: 0.0);
+      unawaited(changeController.forward(from: 0.0));
     });
   }
 
@@ -237,7 +237,7 @@ abstract class LayoutExplorerWidgetState<
         if (status == AnimationStatus.dismissed) {
           setState(() {
             _previousProperties = null;
-            entranceController.forward();
+            unawaited(entranceController.forward());
           });
         }
       });

packages/devtools_app/lib/src/screens/memory/shared/widgets/class_filter.dart

@@ -101,14 +101,16 @@ class _ClassFilterDialogState extends State<ClassFilterDialog> {
     final textFieldLeftPadding = scaleByFontFactor(40.0);
     void onTypeChanged(ClassFilterType? type) => setState(() => _type = type!);
 
-    RadioButton<ClassFilterType> radio(ClassFilterType type, String label) =>
-        RadioButton<ClassFilterType>(
-          label: label,
-          itemValue: type,
-          groupValue: _type,
-          onChanged: onTypeChanged,
-          radioKey: Key(type.toString()),
-        );
+    Widget radio(ClassFilterType type, String label) => Row(
+      children: [
+        Radio<ClassFilterType>(
+          value: type,
+          materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
+          key: Key(type.toString()),
+        ),
+        Expanded(child: Text(label, overflow: TextOverflow.ellipsis)),
+      ],
+    );
 
     Widget textField(TextEditingController controller) => Padding(
       padding: EdgeInsets.only(left: textFieldLeftPadding),
@@ -143,18 +145,22 @@ class _ClassFilterDialogState extends State<ClassFilterDialog> {
         );
         widget.onChanged(newFilter);
       },
-      child: Column(
-        mainAxisSize: MainAxisSize.min,
-        crossAxisAlignment: CrossAxisAlignment.start,
-        children: [
-          radio(ClassFilterType.showAll, 'Show all classes'),
-          const SizedBox(height: defaultSpacing),
-          radio(ClassFilterType.except, 'Show all classes except:'),
-          textField(_except),
-          const SizedBox(height: defaultSpacing),
-          radio(ClassFilterType.only, 'Show only:'),
-          textField(_only),
-        ],
+      child: RadioGroup(
+        groupValue: _type,
+        onChanged: onTypeChanged,
+        child: Column(
+          mainAxisSize: MainAxisSize.min,
+          crossAxisAlignment: CrossAxisAlignment.start,
+          children: [
+            radio(ClassFilterType.showAll, 'Show all classes'),
+            const SizedBox(height: defaultSpacing),
+            radio(ClassFilterType.except, 'Show all classes except:'),
+            textField(_except),
+            const SizedBox(height: defaultSpacing),
+            radio(ClassFilterType.only, 'Show only:'),
+            textField(_only),
+          ],
+        ),
       ),
     );
   }

packages/devtools_app/lib/src/screens/performance/panes/controls/enhance_tracing/enhance_tracing.dart

@@ -346,15 +346,26 @@ class TraceWidgetBuildsScopeSelector extends StatelessWidget {
   Widget build(BuildContext context) {
     final theme = Theme.of(context);
     final textStyle = enabled ? theme.regularTextStyle : theme.subtleTextStyle;
-    return Row(
-      children: [
-        ..._scopeSetting(
-          TraceWidgetBuildsScope.userCreated,
-          textStyle: textStyle,
+    // `Semantics` widget added as a workaround for the bug fixed in
+    // https://github.com/flutter/flutter/pull/170273. This `Semantics` widget
+    // can be removed once that PR is landed and DevTools is updated to use a
+    // version of Flutter with the fix, but it is not an urgent clean up.
+    return Semantics(
+      explicitChildNodes: true,
+      child: RadioGroup<TraceWidgetBuildsScope>(
+        groupValue: scope,
+        onChanged: _changeScope,
+        child: Row(
+          children: [
+            ..._scopeSetting(
+              TraceWidgetBuildsScope.userCreated,
+              textStyle: textStyle,
+            ),
+            const SizedBox(width: defaultSpacing),
+            ..._scopeSetting(TraceWidgetBuildsScope.all, textStyle: textStyle),
+          ],
         ),
-        const SizedBox(width: defaultSpacing),
-        ..._scopeSetting(TraceWidgetBuildsScope.all, textStyle: textStyle),
-      ],
+      ),
     );
   }
 
@@ -364,18 +375,17 @@ class TraceWidgetBuildsScopeSelector extends StatelessWidget {
   }) {
     return [
       Radio<TraceWidgetBuildsScope>(
+        enabled: enabled,
         value: type,
-        groupValue: scope,
         materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
-        onChanged: enabled ? _changeScope : null,
       ),
       Text(type.radioDisplay, style: textStyle),
     ];
   }
 
   Future<void> _changeScope(TraceWidgetBuildsScope? type) async {
-    assert(enabled);
-    final extension = type!.extensionForScope;
+    if (type == null) return;
+    final extension = type.extensionForScope;
     final opposite = type.opposite.extensionForScope;
     await [
       serviceConnection.serviceManager.serviceExtensionManager

packages/devtools_app/lib/src/shared/primitives/collapsible_mixin.dart

@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file or at https://developers.google.com/open-source/licenses/bsd.
 
+import 'dart:async';
+
 import 'package:devtools_app_shared/ui.dart';
 import 'package:flutter/material.dart';
 
@@ -58,9 +60,9 @@ mixin CollapsibleAnimationMixin<T extends StatefulWidget>
   void setExpanded(bool expanded) {
     setState(() {
       if (expanded) {
-        expandController.forward();
+        unawaited(expandController.forward());
       } else {
-        expandController.reverse();
+        unawaited(expandController.reverse());
       }
       onExpandChanged(expanded);
     });
@@ -70,9 +72,9 @@ mixin CollapsibleAnimationMixin<T extends StatefulWidget>
   void didUpdateWidget(Widget oldWidget) {
     super.didUpdateWidget(oldWidget as T);
     if (isExpanded) {
-      expandController.forward();
+      unawaited(expandController.forward());
     } else {
-      expandController.reverse();
+      unawaited(expandController.reverse());
     }
   }
 }

packages/devtools_app/lib/src/shared/ui/common_widgets.dart

@@ -2091,39 +2091,6 @@ class DownloadButton extends StatelessWidget {
   }
 }
 
-class RadioButton<T> extends StatelessWidget {
-  const RadioButton({
-    super.key,
-    required this.label,
-    required this.itemValue,
-    required this.groupValue,
-    this.onChanged,
-    this.radioKey,
-  });
-
-  final String label;
-  final T itemValue;
-  final T groupValue;
-  final void Function(T?)? onChanged;
-  final Key? radioKey;
-
-  @override
-  Widget build(BuildContext context) {
-    return Row(
-      children: [
-        Radio<T>(
-          value: itemValue,
-          groupValue: groupValue,
-          materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
-          onChanged: onChanged,
-          key: radioKey,
-        ),
-        Expanded(child: Text(label, overflow: TextOverflow.ellipsis)),
-      ],
-    );
-  }
-}
-
 class ContextMenuButton extends StatelessWidget {
   ContextMenuButton({
     super.key,

packages/devtools_app/lib/src/shared/ui/side_panel.dart

@@ -76,9 +76,9 @@ class SidePanelViewerState extends State<SidePanelViewer>
       setState(() {
         isVisible = widget.controller.isVisible.value;
         if (isVisible) {
-          visibilityController.forward();
+          unawaited(visibilityController.forward());
         } else {
-          visibilityController.reverse();
+          unawaited(visibilityController.reverse());
         }
       });
     });

packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/property_editor_view.dart

@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file or at https://developers.google.com/open-source/licenses/bsd.
 
+import 'dart:async';
+
 import 'package:devtools_app_shared/ui.dart';
 import 'package:devtools_app_shared/utils.dart';
 import 'package:flutter/material.dart';
@@ -548,9 +550,9 @@ class _ExpandableWidgetDocumentationState
     setState(() {
       _isExpanded = !_isExpanded;
       if (_isExpanded) {
-        _expandAnimationController.forward();
+        unawaited(_expandAnimationController.forward());
       } else {
-        _expandAnimationController.reverse();
+        unawaited(_expandAnimationController.reverse());
       }
     });
   }