Fix trailing commas.

Author: a-siva

dwds/analysis_options.yaml

@@ -10,4 +10,3 @@ analyzer:
 linter:
   rules:
     - always_use_package_imports
-    - require_trailing_commas

dwds/lib/src/services/chrome_proxy_service.dart

@@ -224,7 +224,7 @@ class ChromeProxyService implements VmServiceInterface {
         // Expression evaluation is ready after dependencies are updated.
         if (!_compilerCompleter.isCompleted) _compilerCompleter.complete();
         return result;
-      }, (result) => DwdsEvent.compilerUpdateDependencies(entrypoint),);
+      }, (result) => DwdsEvent.compilerUpdateDependencies(entrypoint));
     }
   }
 
@@ -683,7 +683,7 @@ class ChromeProxyService implements VmServiceInterface {
         RPCErrorKind.kInvalidRequest.code,
         'Expression evaluation is not supported for this configuration.',
       );
-    }, (result) => DwdsEvent.evaluate(expression, result),);
+    }, (result) => DwdsEvent.evaluate(expression, result));
   }
 
   String _newVariableForScope(Map<String, String>? scope) {
@@ -744,7 +744,7 @@ class ChromeProxyService implements VmServiceInterface {
         RPCErrorKind.kInvalidRequest.code,
         'Expression evaluation is not supported for this configuration.',
       );
-    }, (result) => DwdsEvent.evaluateInFrame(expression, result),);
+    }, (result) => DwdsEvent.evaluateInFrame(expression, result));
   }
 
   @override
@@ -796,7 +796,7 @@ class ChromeProxyService implements VmServiceInterface {
       await isInitialized;
       _checkIsolate('getIsolate', isolateId);
       return inspector.isolate;
-    }, (result) => DwdsEvent.getIsolate(),);
+    }, (result) => DwdsEvent.getIsolate());
   }
 
   @override
@@ -848,7 +848,7 @@ class ChromeProxyService implements VmServiceInterface {
       await isInitialized;
       _checkIsolate('getScripts', isolateId);
       return inspector.getScripts();
-    }, (result) => DwdsEvent.getScripts(),);
+    }, (result) => DwdsEvent.getScripts());
   }
 
   @override
@@ -900,7 +900,7 @@ class ChromeProxyService implements VmServiceInterface {
         reportLines: reportLines,
         libraryFilters: libraryFilters,
       );
-    }, (result) => DwdsEvent.getSourceReport(),);
+    }, (result) => DwdsEvent.getSourceReport());
   }
 
   /// Returns the current stack.
@@ -936,7 +936,7 @@ class ChromeProxyService implements VmServiceInterface {
     return captureElapsedTime(() async {
       await isInitialized;
       return _vm;
-    }, (result) => DwdsEvent.getVM(),);
+    }, (result) => DwdsEvent.getVM());
   }
 
   @override
@@ -1159,7 +1159,7 @@ class ChromeProxyService implements VmServiceInterface {
           step: step,
           frameIndex: frameIndex,
         );
-      }, (result) => DwdsEvent.resume(step),);
+      }, (result) => DwdsEvent.resume(step));
     } else {
       inspector.appConnection.runMain();
       return Success();

dwds/lib/src/utilities/server.dart

@@ -68,7 +68,7 @@ void serveHttpRequests(
 ) {
   return Chain.capture(() {
     serveRequests(requests, handler);
-  }, onError: onError,);
+  }, onError: onError);
 }
 
 /// Throws an [wip.ExceptionDetails] object if `exceptionDetails` is present on the
@@ -96,7 +96,7 @@ Map<String, dynamic> getResultOrHandleError(
   if (result == null) {
     throw ChromeDebugException({
       'text': 'null result from Chrome Devtools',
-    }, evalContents: evalContents,);
+    }, evalContents: evalContents);
   }
   return result;
 }

dwds/lib/src/utilities/shared.dart

@@ -44,5 +44,5 @@ Future<T> wrapInErrorHandlerAsync<T>(
         'Unexpected DWDS error for $command: $error',
       ),
     );
-  }, test: (e) => e is! RPCError && e is! SentinelException,);
+  }, test: (e) => e is! RPCError && e is! SentinelException);
 }

dwds/test/common/chrome_proxy_service_common.dart

@@ -1462,7 +1462,7 @@ void runTests({
 
         final sourceReport = await service.getSourceReport(isolateId, [
           'PossibleBreakpoints',
-        ], scriptId: mainScript.id,);
+        ], scriptId: mainScript.id);
 
         expect(sourceReport.scripts, isNotEmpty);
         expect(sourceReport.ranges, isNotEmpty);

dwds/test/devtools_test.dart

@@ -54,7 +54,7 @@ void main() {
       expect(await context.webDriver.pageSource, contains('DevTools'));
       expect(await context.webDriver.currentUrl, contains('ide=Dwds'));
       // TODO(https://github.com/dart-lang/webdev/issues/1888): Re-enable.
-    }, skip: Platform.isWindows,);
+    }, skip: Platform.isWindows);
 
     test(
       'can not launch devtools for the same app in multiple tabs',
@@ -140,7 +140,7 @@ void main() {
       },
       skip: 'https://github.com/dart-lang/webdev/issues/1888',
     );
-  }, timeout: Timeout.factor(2),);
+  }, timeout: Timeout.factor(2));
 
   group('Injected client without a DevTools server', () {
     setUp(() async {

dwds/test/evaluate_common.dart

@@ -612,7 +612,7 @@ void testAll({
             matchErrorRef(contains(EvaluationErrorKind.loadModule)),
           );
         });
-      }, skip: 'https://github.com/dart-lang/sdk/issues/48587',);
+      }, skip: 'https://github.com/dart-lang/sdk/issues/48587');
 
       test('cannot evaluate in unsupported isolate', () async {
         await onBreakPoint(mainScript, 'printLocal', (event) async {
@@ -795,7 +795,7 @@ void testAll({
         expect(result, matchInstanceRef('42'));
       });
     });
-  }, timeout: const Timeout.factor(2),);
+  }, timeout: const Timeout.factor(2));
 
   group('shared context with no evaluation |', () {
     setUpAll(() async {

dwds/test/events_test.dart

@@ -164,7 +164,7 @@ void main() {
               'elapsedMilliseconds': isNotNull,
               'screen': equals('debugger'),
             }),
-          ], () => keyboard.sendChord([Keyboard.alt, 'd']),);
+          ], () => keyboard.sendChord([Keyboard.alt, 'd']));
         },
         skip: 'https://github.com/dart-lang/webdev/issues/2394',
       );
@@ -380,7 +380,7 @@ void main() {
             }),
             () => service.getSourceReport(isolateId, [
               SourceReportKind.kPossibleBreakpoints,
-            ], scriptId: mainScript.id,),
+            ], scriptId: mainScript.id),
           );
         });
       });

dwds/test/fixtures/context.dart

@@ -209,7 +209,7 @@ class TestContext {
       await Process.run(sdkLayout.dartPath, [
         'pub',
         'upgrade',
-      ], workingDirectory: project.absolutePackageDirectory,);
+      ], workingDirectory: project.absolutePackageDirectory);
 
       ExpressionCompiler? expressionCompiler;
       AssetReader assetReader;

dwds/test/fixtures/project.dart

@@ -156,7 +156,7 @@ class TestProject {
       'run',
       'build_runner',
       'clean',
-    ], workingDirectory: absolutePackageDirectory,);
+    ], workingDirectory: absolutePackageDirectory);
   }
 
   /// The path to the Dart specified file in the 'lib' directory, e.g,