[web] enable always_specify_types lint (#27406)

Author: yjbanov

lib/web_ui/analysis_options.yaml

@@ -13,7 +13,6 @@ analyzer:
 
 linter:
   rules:
-    always_specify_types: false
     annotate_overrides: false
     avoid_classes_with_only_static_members: false
     avoid_empty_else: false

lib/web_ui/dev/browser.dart

@@ -76,20 +76,20 @@ abstract class Browser {
   ///
   /// This will fire once the process has started successfully.
   Future<Process> get _process => _processCompleter.future;
-  final _processCompleter = Completer<Process>();
+  final Completer<Process> _processCompleter = Completer<Process>();
 
   /// Whether [close] has been called.
-  var _closed = false;
+  bool _closed = false;
 
   /// A future that completes when the browser exits.
   ///
   /// If there's a problem starting or running the browser, this will complete
   /// with an error.
   Future<void> get onExit => _onExitCompleter.future;
-  final _onExitCompleter = Completer<void>();
+  final Completer<void> _onExitCompleter = Completer<void>();
 
   /// Standard IO streams for the underlying browser process.
-  final _ioSubscriptions = <StreamSubscription>[];
+  final List<StreamSubscription<void>> _ioSubscriptions = <StreamSubscription<void>>[];
 
   /// Creates a new browser.
   ///
@@ -102,10 +102,10 @@ abstract class Browser {
     // for the process to actually start. They should just wait for the HTTP
     // request instead.
     runZonedGuarded(() async {
-      var process = await startBrowser();
+      Process process = await startBrowser();
       _processCompleter.complete(process);
 
-      var output = Uint8Buffer();
+      Uint8Buffer output = Uint8Buffer();
       void drainOutput(Stream<List<int>> stream) {
         try {
           _ioSubscriptions
@@ -117,7 +117,7 @@ abstract class Browser {
       drainOutput(process.stdout);
       drainOutput(process.stderr);
 
-      var exitCode = await process.exitCode;
+      int exitCode = await process.exitCode;
 
       // This hack dodges an otherwise intractable race condition. When the user
       // presses Control-C, the signal is sent to the browser and the test
@@ -134,8 +134,8 @@ abstract class Browser {
       }
 
       if (!_closed && exitCode != 0) {
-        var outputString = utf8.decode(output);
-        var message = '$name failed with exit code $exitCode.';
+        String outputString = utf8.decode(output);
+        String message = '$name failed with exit code $exitCode.';
         if (outputString.isNotEmpty) {
           message += '\nStandard output:\n$outputString';
         }
@@ -151,7 +151,7 @@ abstract class Browser {
       }
 
       // Make sure the process dies even if the error wasn't fatal.
-      _process.then((process) => process.kill());
+      _process.then((Process process) => process.kill());
 
       if (stackTrace == null) {
         stackTrace = Trace.current();
@@ -170,13 +170,13 @@ abstract class Browser {
   ///
   /// Returns the same [Future] as [onExit], except that it won't emit
   /// exceptions.
-  Future close() async {
+  Future<void> close() async {
     _closed = true;
 
     // If we don't manually close the stream the test runner can hang.
     // For example this happens with Chrome Headless.
     // See SDK issue: https://github.com/dart-lang/sdk/issues/31264
-    for (var stream in _ioSubscriptions) {
+    for (StreamSubscription<void> stream in _ioSubscriptions) {
       unawaited(stream.cancel());
     }
 
@@ -192,7 +192,7 @@ abstract class ScreenshotManager {
   /// Capture a screenshot.
   ///
   /// Please read the details for the implementing classes.
-  Future<Image> capture(math.Rectangle region);
+  Future<Image> capture(math.Rectangle<num> region);
 
   /// Suffix to be added to the end of the filename.
   ///

lib/web_ui/dev/build.dart

@@ -7,12 +7,13 @@ import 'dart:async';
 
 import 'package:args/command_runner.dart';
 import 'package:path/path.dart' as path;
+import 'package:watcher/src/watch_event.dart';
 
 import 'environment.dart';
 import 'utils.dart';
 import 'watcher.dart';
 
-class BuildCommand extends Command<bool> with ArgUtils {
+class BuildCommand extends Command<bool> with ArgUtils<bool> {
   BuildCommand() {
     argParser
       ..addFlag(
@@ -48,7 +49,7 @@ class BuildCommand extends Command<bool> with ArgUtils {
         dir: libPath.absolute,
         pipeline: buildPipeline,
         // Ignore font files that are copied whenever tests run.
-        ignore: (event) => event.path.endsWith('.ttf'),
+        ignore: (WatchEvent event) => event.path.endsWith('.ttf'),
       ).start();
     }
     return true;

lib/web_ui/dev/chrome.dart

@@ -51,7 +51,7 @@ class ChromeEnvironment implements BrowserEnvironment {
 /// Any errors starting or running the process are reported through [onExit].
 class Chrome extends Browser {
   @override
-  final name = 'Chrome';
+  final String name = 'Chrome';
 
   @override
   final Future<Uri> remoteDebuggerUrl;
@@ -60,7 +60,7 @@ class Chrome extends Browser {
   /// [Uri] or a [String].
   factory Chrome(Uri url, {bool debug = false}) {
     String version = ChromeArgParser.instance.version;
-    var remoteDebuggerCompleter = Completer<Uri>.sync();
+    Completer<Uri> remoteDebuggerCompleter = Completer<Uri>.sync();
     return Chrome._(() async {
       final BrowserInstallation installation = await getOrInstallChrome(
         version,
@@ -80,7 +80,7 @@ class Chrome extends Browser {
       final bool isChromeNoSandbox =
           Platform.environment['CHROME_NO_SANDBOX'] == 'true';
       final String dir = environment.webUiDartToolDir.createTempSync('test_chrome_user_data_').resolveSymbolicLinksSync();
-      var args = [
+      List<String> args = <String>[
         '--user-data-dir=$dir',
         url.toString(),
         if (!debug)
@@ -218,7 +218,7 @@ class ChromeScreenshotManager extends ScreenshotManager {
   /// [region] is used to decide which part of the web content will be used in
   /// test image. It includes starting coordinate x,y as well as height and
   /// width of the area to capture.
-  Future<Image> capture(math.Rectangle? region) async {
+  Future<Image> capture(math.Rectangle<num>? region) async {
     final wip.ChromeConnection chromeConnection =
         wip.ChromeConnection('localhost', kDevtoolsPort);
     final wip.ChromeTab? chromeTab = await chromeConnection.getTab(

lib/web_ui/dev/chrome_installer.dart

@@ -4,6 +4,7 @@
 
 import 'dart:async';
 import 'dart:io' as io;
+import 'dart:typed_data';
 
 import 'package:archive/archive.dart';
 import 'package:archive/archive_io.dart';
@@ -220,15 +221,15 @@ class ChromeInstaller {
       final Stopwatch stopwatch = Stopwatch()..start();
 
       // Read the Zip file from disk.
-      final bytes = downloadedFile.readAsBytesSync();
+      final Uint8List bytes = downloadedFile.readAsBytesSync();
 
       final Archive archive = ZipDecoder().decodeBytes(bytes);
 
       // Extract the contents of the Zip archive to disk.
       for (final ArchiveFile file in archive) {
         final String filename = file.name;
         if (file.isFile) {
-          final data = file.content as List<int>;
+          final List<int> data = file.content as List<int>;
           io.File(path.join(versionDir.path, filename))
             ..createSync(recursive: true)
             ..writeAsBytesSync(data);

lib/web_ui/dev/clean.dart

@@ -11,7 +11,7 @@ import 'package:path/path.dart' as path;
 import 'environment.dart';
 import 'utils.dart';
 
-class CleanCommand extends Command<bool> with ArgUtils {
+class CleanCommand extends Command<bool> with ArgUtils<bool> {
   CleanCommand() {
     argParser
       ..addFlag(

lib/web_ui/dev/common.dart

@@ -221,7 +221,7 @@ class DevNull implements StringSink {
   void write(Object? obj) {}
 
   @override
-  void writeAll(Iterable objects, [String separator = ""]) {}
+  void writeAll(Iterable<dynamic> objects, [String separator = ""]) {}
 
   @override
   void writeCharCode(int charCode) {}

lib/web_ui/dev/create_simulator.dart

@@ -10,7 +10,7 @@ import 'package:simulators/simulator_manager.dart';
 import 'safari_installation.dart';
 import 'utils.dart';
 
-class CreateSimulatorCommand extends Command<bool> with ArgUtils {
+class CreateSimulatorCommand extends Command<bool> with ArgUtils<bool> {
   CreateSimulatorCommand() {
     IosSafariArgParser.instance.populateOptions(argParser);
     argParser

lib/web_ui/dev/edge.dart

@@ -42,7 +42,7 @@ class EdgeEnvironment implements BrowserEnvironment {
 /// Any errors starting or running the process are reported through [onExit].
 class Edge extends Browser {
   @override
-  final name = 'Edge';
+  final String name = 'Edge';
 
   /// Starts a new instance of Safari open to the given [url], which may be a
   /// [Uri] or a [String].
@@ -63,8 +63,10 @@ class Edge extends Browser {
         pathToOpen = pathToOpen.substring(0, index-1);
       }
 
-      var process =
-          await Process.start(installation.executable, ['$pathToOpen','-k']);
+      Process process = await Process.start(
+        installation.executable,
+        <String>['$pathToOpen','-k'],
+      );
 
       return process;
     });

lib/web_ui/dev/felt.dart

@@ -14,7 +14,7 @@ import 'exceptions.dart';
 import 'test_runner.dart';
 import 'utils.dart';
 
-CommandRunner runner = CommandRunner<bool>(
+CommandRunner<bool> runner = CommandRunner<bool>(
   'felt',
   'Command-line utility for building and testing Flutter web engine.',
 )
@@ -26,7 +26,7 @@ CommandRunner runner = CommandRunner<bool>(
 
 void main(List<String> rawArgs) async {
   // Remove --clean from the list as that's processed by the wrapper script.
-  final List<String> args = rawArgs.where((arg) => arg != '--clean').toList();
+  final List<String> args = rawArgs.where((String arg) => arg != '--clean').toList();
 
   if (args.isEmpty) {
     // The felt tool was invoked with no arguments. Print usage.