Bump flutter candidate and update formatting. (#9165)

Author: kenzieschmoll

.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.24.2-1 # To avoid errors add `-1` (build number) to the version
+          sudo apt-get install dcm=1.28.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 @@
-a7f820163c5d7d5321872c60f22fa047fb94bd7b
+80d0409d137acdd55af4124b7622915c062c9d21

packages/devtools_app/benchmark/devtools_benchmarks_test.dart

@@ -53,10 +53,9 @@ Future<void> _runBenchmarks({bool useWasm = false}) async {
   final taskResult = await serveWebBenchmark(
     benchmarkAppDirectory: projectRootDirectory(),
     entryPoint: generateBenchmarkEntryPoint(useWasm: useWasm),
-    compilationOptions:
-        useWasm
-            ? const CompilationOptions.wasm()
-            : const CompilationOptions.js(),
+    compilationOptions: useWasm
+        ? const CompilationOptions.wasm()
+        : const CompilationOptions.js(),
     treeShakeIcons: false,
     benchmarkPath: benchmarkPath(useWasm: useWasm),
   );

packages/devtools_app/benchmark/scripts/run_benchmarks.dart

@@ -56,10 +56,9 @@ Future<BenchmarkResults> runBenchmarks({
       await serveWebBenchmark(
         benchmarkAppDirectory: projectRootDirectory(),
         entryPoint: generateBenchmarkEntryPoint(useWasm: useWasm),
-        compilationOptions:
-            useWasm
-                ? const CompilationOptions.wasm()
-                : const CompilationOptions.js(),
+        compilationOptions: useWasm
+            ? const CompilationOptions.wasm()
+            : const CompilationOptions.js(),
         treeShakeIcons: false,
         benchmarkPath: benchmarkPath(useWasm: useWasm),
         headless: !useBrowser,

packages/devtools_app/benchmark/test_infra/client/client_shared.dart

@@ -10,16 +10,11 @@ import '../devtools_recorder.dart';
 typedef RecorderFactory = Recorder Function();
 
 final benchmarks = <String, RecorderFactory>{
-  DevToolsBenchmark.navigateThroughOfflineScreens.id:
-      () => DevToolsRecorder(
-        benchmark: DevToolsBenchmark.navigateThroughOfflineScreens,
-      ),
-  DevToolsBenchmark.offlineCpuProfilerScreen.id:
-      () => DevToolsRecorder(
-        benchmark: DevToolsBenchmark.offlineCpuProfilerScreen,
-      ),
-  DevToolsBenchmark.offlinePerformanceScreen.id:
-      () => DevToolsRecorder(
-        benchmark: DevToolsBenchmark.offlinePerformanceScreen,
-      ),
+  DevToolsBenchmark.navigateThroughOfflineScreens.id: () => DevToolsRecorder(
+    benchmark: DevToolsBenchmark.navigateThroughOfflineScreens,
+  ),
+  DevToolsBenchmark.offlineCpuProfilerScreen.id: () =>
+      DevToolsRecorder(benchmark: DevToolsBenchmark.offlineCpuProfilerScreen),
+  DevToolsBenchmark.offlinePerformanceScreen.id: () =>
+      DevToolsRecorder(benchmark: DevToolsBenchmark.offlinePerformanceScreen),
 };

packages/devtools_app/benchmark/test_infra/common.dart

@@ -12,11 +12,10 @@ const _benchmarkInitialPage = '';
 
 const _wasmQueryParameters = {'wasm': 'true'};
 
-String benchmarkPath({required bool useWasm}) =>
-    Uri(
-      path: _benchmarkInitialPage,
-      queryParameters: useWasm ? _wasmQueryParameters : null,
-    ).toString();
+String benchmarkPath({required bool useWasm}) => Uri(
+  path: _benchmarkInitialPage,
+  queryParameters: useWasm ? _wasmQueryParameters : null,
+).toString();
 
 String generateBenchmarkEntryPoint({required bool useWasm}) {
   return 'benchmark/test_infra/client/client_${useWasm ? 'wasm' : 'js'}.dart';

packages/devtools_app/integration_test/run_tests.dart

@@ -55,8 +55,8 @@ void main(List<String> args) async {
     testRunnerArgs: testRunnerArgs,
     runTest: _runTest,
     newArgsGenerator: (args) => DevToolsAppTestRunnerArgs(args),
-    testIsSupported:
-        (testFile) => testRunnerArgs.testAppDevice.supportsTest(testFile.path),
+    testIsSupported: (testFile) =>
+        testRunnerArgs.testAppDevice.supportsTest(testFile.path),
     debugLogging: debugTestScript,
   );
 }

packages/devtools_app/integration_test/test/live_connection/devtools_extensions_test.dart

@@ -268,11 +268,10 @@ Future<void> _verifyExtensionsSettingsMenu(
   await _openExtensionSettingsMenu(tester);
 
   expect(find.byType(ExtensionSetting), findsNWidgets(enabledStates.length));
-  final toggleButtonGroups =
-      tester
-          .widgetList(find.byType(DevToolsToggleButtonGroup))
-          .cast<DevToolsToggleButtonGroup>()
-          .toList();
+  final toggleButtonGroups = tester
+      .widgetList(find.byType(DevToolsToggleButtonGroup))
+      .cast<DevToolsToggleButtonGroup>()
+      .toList();
   for (int i = 0; i < toggleButtonGroups.length; i++) {
     logStatus('verify extension settings toggle button states (index $i)');
     final group = toggleButtonGroups[i];
@@ -313,11 +312,10 @@ Future<void> _changeExtensionSetting(
     'changing the extension setting at index $extensionIndex to value $settingValue',
   );
   await _openExtensionSettingsMenu(tester);
-  final extensionSetting =
-      tester
-          .widgetList(find.byType(DevToolsToggleButtonGroup))
-          .cast<DevToolsToggleButtonGroup>()
-          .toList()[extensionIndex];
+  final extensionSetting = tester
+      .widgetList(find.byType(DevToolsToggleButtonGroup))
+      .cast<DevToolsToggleButtonGroup>()
+      .toList()[extensionIndex];
   await tester.tap(
     find.descendant(
       of: find.byWidget(extensionSetting),

packages/devtools_app/integration_test/test_infra/run/_test_app_driver.dart

@@ -133,10 +133,9 @@ class TestFlutterApp extends IntegrationTestApp {
     return _timeoutWithMessages<Map<String, Object?>>(
       () => response.future,
       timeout: timeout,
-      message:
-          event != null
-              ? 'Did not receive expected $event event.'
-              : 'Did not receive response to request "$id".',
+      message: event != null
+          ? 'Did not receive expected $event event.'
+          : 'Did not receive response to request "$id".',
     ).whenComplete(() => sub.cancel());
   }
 
@@ -350,8 +349,8 @@ abstract class IntegrationTestApp with IOMixin {
     _debugPrint('Waiting for process to end');
     return runProcess!.exitCode.timeout(
       IOMixin.killTimeout,
-      onTimeout:
-          () => killGracefully(runProcess!, debugLogging: debugTestScript),
+      onTimeout: () =>
+          killGracefully(runProcess!, debugLogging: debugTestScript),
     );
   }
 
@@ -387,8 +386,9 @@ abstract class IntegrationTestApp with IOMixin {
 
   String _debugPrint(String msg) {
     const maxLength = 500;
-    final truncatedMsg =
-        msg.length > maxLength ? '${msg.substring(0, maxLength)}...' : msg;
+    final truncatedMsg = msg.length > maxLength
+        ? '${msg.substring(0, maxLength)}...'
+        : msg;
     _allMessages.add(truncatedMsg);
     debugLog('_TestApp - $truncatedMsg');
     return msg;
@@ -404,12 +404,11 @@ Uri convertToWebSocketUrl({required Uri serviceProtocolUrl}) {
       serviceProtocolUrl.isScheme('https');
   final scheme = isSecure ? 'wss' : 'ws';
 
-  final path =
-      serviceProtocolUrl.path.endsWith('/ws')
-          ? serviceProtocolUrl.path
-          : (serviceProtocolUrl.path.endsWith('/')
-              ? '${serviceProtocolUrl.path}ws'
-              : '${serviceProtocolUrl.path}/ws');
+  final path = serviceProtocolUrl.path.endsWith('/ws')
+      ? serviceProtocolUrl.path
+      : (serviceProtocolUrl.path.endsWith('/')
+            ? '${serviceProtocolUrl.path}ws'
+            : '${serviceProtocolUrl.path}/ws');
 
   return serviceProtocolUrl.replace(scheme: scheme, path: path);
 }

packages/devtools_app/integration_test/test_infra/run/run_test.dart

@@ -89,11 +89,10 @@ Future<void> runFlutterIntegrationTest(
 class DevToolsAppTestRunnerArgs extends IntegrationTestRunnerArgs {
   DevToolsAppTestRunnerArgs(super.args, {super.verifyValidTarget = true})
     : super(addExtraArgs: _addExtraArgs) {
-    testAppDevice =
-        TestAppDevice.fromArgName(
-          argResults.option(_testAppDeviceArg) ??
-              TestAppDevice.flutterTester.argName,
-        )!;
+    testAppDevice = TestAppDevice.fromArgName(
+      argResults.option(_testAppDeviceArg) ??
+          TestAppDevice.flutterTester.argName,
+    )!;
   }
 
   /// The type of device for the test app to run on.