Revert "implicit-casts: false (#47199)" (#47428)

This reverts commit c235998a.

analysis_options.yaml

@@ -20,7 +20,6 @@
 
 analyzer:
   strong-mode:
-    implicit-casts: false
     implicit-dynamic: false
   errors:
     # treat missing required parameters as a warning (not a hint)

dev/devicelab/bin/tasks/fast_scroll_large_images__memory.dart

@@ -20,7 +20,7 @@ class FastScrollLargeImagesMemoryTest extends MemoryTest {
         );
 
   @override
-  AndroidDevice get device => super.device as AndroidDevice;
+  AndroidDevice get device => super.device;
 
   @override
   int get iterationCount => 5;

dev/integration_tests/flutter_driver_screenshot_test/test_driver/driver_screenshot_tester.dart

@@ -4,11 +4,10 @@
 
 import 'dart:async';
 import 'dart:io' show File;
-
-import 'package:flutter/foundation.dart';
-import 'package:flutter_driver/flutter_driver.dart';
 import 'package:meta/meta.dart';
 import 'package:path/path.dart' as path;
+import 'package:flutter_driver/flutter_driver.dart';
+import 'package:collection/collection.dart';
 
 const String _kPathParent = 'test_driver/goldens/';
 
@@ -44,7 +43,8 @@ class DriverScreenShotTester {
   Future<bool> compareScreenshots(List<int> screenshot) async {
     final File file = File(_getImageFilePath());
     final List<int> matcher = await file.readAsBytes();
-    return listEquals<int>(screenshot, matcher);
+    final Function listEquals = const ListEquality<int>().equals;
+    return listEquals(screenshot, matcher);
   }
 
   /// Returns a bytes representation of a screenshot on the current screen.

dev/tools/localization/bin/gen_l10n.dart

@@ -53,10 +53,10 @@ Future<void> main(List<String> arguments) async {
     exit(0);
   }
 
-  final String arbPathString = results['arb-dir'] as String;
-  final String outputFileString = results['output-localization-file'] as String;
-  final String templateArbFileName = results['template-arb-file'] as String;
-  final String classNameString = results['output-class'] as String;
+  final String arbPathString = results['arb-dir'];
+  final String outputFileString = results['output-localization-file'];
+  final String templateArbFileName = results['template-arb-file'];
+  final String classNameString = results['output-class'];
 
   const local.LocalFileSystem fs = local.LocalFileSystem();
   final LocalizationsGenerator localizationsGenerator = LocalizationsGenerator(fs);

dev/tools/localization/gen_l10n.dart

@@ -599,8 +599,8 @@ class LocalizationsGenerator {
     for (File file in fileSystemEntityList) {
       final String filePath = file.path;
       if (arbFilenameRE.hasMatch(filePath)) {
-        final Map<String, dynamic> arbContents = json.decode(file.readAsStringSync()) as Map<String, dynamic>;
-        String localeString = arbContents['@@locale'] as String;
+        final Map<String, dynamic> arbContents = json.decode(file.readAsStringSync());
+        String localeString = arbContents['@@locale'];
         if (localeString == null) {
           final RegExpMatch arbFileMatch = arbFilenameLocaleRE.firstMatch(filePath);
           if (arbFileMatch == null) {
@@ -653,7 +653,7 @@ class LocalizationsGenerator {
   void generateClassMethods() {
     Map<String, dynamic> bundle;
     try {
-      bundle = json.decode(templateArbFile.readAsStringSync()) as Map<String, dynamic>;
+      bundle = json.decode(templateArbFile.readAsStringSync());
     } on FileSystemException catch (e) {
       throw FileSystemException('Unable to read input arb file: $e');
     } on FormatException catch (e) {
@@ -669,7 +669,7 @@ class LocalizationsGenerator {
           'Invalid key format: $key \n It has to be in camel case, cannot start '
           'with a number, and cannot contain non-alphanumeric characters.'
         );
-      if (pluralValueRE.hasMatch(bundle[key] as String))
+      if (pluralValueRE.hasMatch(bundle[key]))
         classMethods.add(genPluralMethod(bundle, key));
       else
         classMethods.add(genSimpleMethod(bundle, key));

packages/flutter/lib/src/gestures/mouse_tracking.dart

@@ -314,7 +314,7 @@ class MouseTracker extends ChangeNotifier {
     final int device = state.device;
     return (_mouseStates.containsKey(device) && _trackedAnnotations.isNotEmpty)
       ? LinkedHashSet<MouseTrackerAnnotation>.from(annotationFinder(globalPosition))
-      : <MouseTrackerAnnotation>{} as LinkedHashSet<MouseTrackerAnnotation>;
+      : <MouseTrackerAnnotation>{};
   }
 
   static bool get _duringBuildPhase {

packages/flutter/lib/src/material/reorderable_list.dart

@@ -530,7 +530,7 @@ class _ReorderableListContentState extends State<_ReorderableListContent> with T
           return _dragging == toAccept && toAccept != toWrap.key;
         },
         onAccept: (Key accepted) { },
-        onLeave: (Object leaving) { },
+        onLeave: (Key leaving) { },
       );
     });
   }

packages/flutter/lib/src/painting/edge_insets.dart

@@ -162,12 +162,12 @@ abstract class EdgeInsetsGeometry {
   /// or equal to `min`, and less than or equal to `max`.
   EdgeInsetsGeometry clamp(EdgeInsetsGeometry min, EdgeInsetsGeometry max) {
     return _MixedEdgeInsets.fromLRSETB(
-      _left.clamp(min._left, max._left) as double,
-      _right.clamp(min._right, max._right) as double,
-      _start.clamp(min._start, max._start) as double,
-      _end.clamp(min._end, max._end) as double,
-      _top.clamp(min._top, max._top) as double,
-      _bottom.clamp(min._bottom, max._bottom) as double,
+      _left.clamp(min._left, max._left),
+      _right.clamp(min._right, max._right),
+      _start.clamp(min._start, max._start),
+      _end.clamp(min._end, max._end),
+      _top.clamp(min._top, max._top),
+      _bottom.clamp(min._bottom, max._bottom),
     );
   }
 

packages/flutter/lib/src/rendering/layer.dart

@@ -672,9 +672,9 @@ class PlatformViewLayer extends Layer {
     if (hoverAnnotation == null || !rect.contains(localPosition)) {
       return false;
     }
-    if (S == MouseTrackerAnnotation) {
+    if (MouseTrackerAnnotation == S) {
       final Object untypedValue = hoverAnnotation;
-      final S typedValue = untypedValue as S;
+      final S typedValue = untypedValue;
       result.add(AnnotationEntry<S>(
         annotation: typedValue,
         localPosition: localPosition,

packages/flutter/lib/src/rendering/proxy_sliver.dart

@@ -7,7 +7,6 @@ import 'dart:ui' as ui show Color;
 import 'package:flutter/foundation.dart';
 import 'package:vector_math/vector_math_64.dart';
 
-import 'layer.dart';
 import 'object.dart';
 import 'sliver.dart';
 
@@ -72,7 +71,7 @@ abstract class RenderProxySliver extends RenderSliver with RenderObjectWithChild
   @override
   void applyPaintTransform(RenderObject child, Matrix4 transform) {
     assert(child != null);
-    final SliverPhysicalParentData childParentData = child.parentData as SliverPhysicalParentData;
+    final SliverPhysicalParentData childParentData = child.parentData;
     childParentData.applyPaintTransform(transform);
   }
 }
@@ -170,7 +169,7 @@ class RenderSliverOpacity extends RenderProxySliver {
         offset,
         _alpha,
         _paintWithOpacity,
-        oldLayer: layer as OpacityLayer,
+        oldLayer: layer,
       );
     }
   }

packages/flutter/lib/src/widgets/binding.dart

@@ -332,7 +332,7 @@ mixin WidgetsBinding on BindingBase, ServicesBinding, SchedulerBinding, GestureB
       registerServiceExtension(
         name: 'fastReassemble',
         callback: (Map<String, Object> params) async {
-          final String className = params['class'] as String;
+          final String className = params['class'];
           void markElementsDirty(Element element) {
             if (element == null) {
               return;

packages/flutter/lib/src/widgets/drag_target.dart

@@ -48,7 +48,7 @@ typedef DragEndCallback = void Function(DraggableDetails details);
 /// Signature for when a [Draggable] leaves a [DragTarget].
 ///
 /// Used by [DragTarget.onLeave].
-typedef DragTargetLeave = void Function(Object data);
+typedef DragTargetLeave<T> = void Function(T data);
 
 /// Where the [Draggable] should be anchored during a drag.
 enum DragAnchor {
@@ -502,7 +502,7 @@ class DragTarget<T> extends StatefulWidget {
 
   /// Called when a given piece of data being dragged over this target leaves
   /// the target.
-  final DragTargetLeave onLeave;
+  final DragTargetLeave<T> onLeave;
 
   @override
   _DragTargetState<T> createState() => _DragTargetState<T>();
@@ -514,12 +514,13 @@ List<T> _mapAvatarsToData<T>(List<_DragAvatar<T>> avatars) {
 
 class _DragTargetState<T> extends State<DragTarget<T>> {
   final List<_DragAvatar<T>> _candidateAvatars = <_DragAvatar<T>>[];
-  final List<_DragAvatar<Object>> _rejectedAvatars = <_DragAvatar<Object>>[];
+  final List<_DragAvatar<dynamic>> _rejectedAvatars = <_DragAvatar<dynamic>>[];
 
-  bool didEnter(_DragAvatar<Object> avatar) {
+  bool didEnter(_DragAvatar<dynamic> avatar) {
     assert(!_candidateAvatars.contains(avatar));
     assert(!_rejectedAvatars.contains(avatar));
-    if (avatar is _DragAvatar<T> && (widget.onWillAccept == null || widget.onWillAccept(avatar.data))) {
+    final dynamic data = avatar.data;
+    if (data is T && (widget.onWillAccept == null || widget.onWillAccept(data))) {
       setState(() {
         _candidateAvatars.add(avatar);
       });
@@ -532,7 +533,7 @@ class _DragTargetState<T> extends State<DragTarget<T>> {
     }
   }
 
-  void didLeave(_DragAvatar<Object> avatar) {
+  void didLeave(_DragAvatar<dynamic> avatar) {
     assert(_candidateAvatars.contains(avatar) || _rejectedAvatars.contains(avatar));
     if (!mounted)
       return;
@@ -544,7 +545,7 @@ class _DragTargetState<T> extends State<DragTarget<T>> {
       widget.onLeave(avatar.data);
   }
 
-  void didDrop(_DragAvatar<Object> avatar) {
+  void didDrop(_DragAvatar<dynamic> avatar) {
     assert(_candidateAvatars.contains(avatar));
     if (!mounted)
       return;
@@ -552,7 +553,7 @@ class _DragTargetState<T> extends State<DragTarget<T>> {
       _candidateAvatars.remove(avatar);
     });
     if (widget.onAccept != null)
-      widget.onAccept(avatar.data as T);
+      widget.onAccept(avatar.data);
   }
 
   @override
@@ -561,7 +562,7 @@ class _DragTargetState<T> extends State<DragTarget<T>> {
     return MetaData(
       metaData: this,
       behavior: HitTestBehavior.translucent,
-      child: widget.builder(context, _mapAvatarsToData<T>(_candidateAvatars), _mapAvatarsToData<Object>(_rejectedAvatars)),
+      child: widget.builder(context, _mapAvatarsToData<T>(_candidateAvatars), _mapAvatarsToData<dynamic>(_rejectedAvatars)),
     );
   }
 }

packages/flutter/test/widgets/draggable_test.dart

@@ -90,21 +90,13 @@ void main() {
             builder: (BuildContext context, List<int> data, List<dynamic> rejects) {
               return Container(height: 100.0, child: const Text('Target 1'));
             },
-            onLeave: (Object data) {
-              if (data is int) {
-                leftBehind['Target 1'] = leftBehind['Target 1'] + data;
-              }
-            },
+            onLeave: (int data) => leftBehind['Target 1'] = leftBehind['Target 1'] + data,
           ),
           DragTarget<int>(
             builder: (BuildContext context, List<int> data, List<dynamic> rejects) {
               return Container(height: 100.0, child: const Text('Target 2'));
             },
-            onLeave: (Object data) {
-              if (data is int) {
-                leftBehind['Target 2'] = leftBehind['Target 2'] + data;
-              }
-            },
+            onLeave: (int data) => leftBehind['Target 2'] = leftBehind['Target 2'] + data,
           ),
         ],
       ),

packages/flutter_driver/analysis_options.yaml

+# Override the parent analysis_options until all the repo has implicit-casts: false
+
+include: ../analysis_options.yaml
+
+analyzer:
+  strong-mode:
+    implicit-casts: false
+    implicit-dynamic: false

packages/flutter_goldens/analysis_options.yaml

@@ -2,3 +2,8 @@
 # the ones from above, which include the `public_member_api_docs` rule).
 
 include: ../../analysis_options.yaml
+
+analyzer:
+  strong-mode:
+    implicit-casts: false
+    implicit-dynamic: false

packages/flutter_goldens_client/analysis_options.yaml

@@ -2,3 +2,8 @@
 # the ones from above, which include the `public_member_api_docs` rule).
 
 include: ../../analysis_options.yaml
+
+analyzer:
+  strong-mode:
+    implicit-casts: false
+    implicit-dynamic: false

packages/flutter_localizations/analysis_options.yaml

+# Override the parent analysis_options until all the repo has implicit-casts: false
+
+include: ../analysis_options.yaml
+
+analyzer:
+  strong-mode:
+    implicit-casts: false
+    implicit-dynamic: false
+
+linter:
+  rules:
+    avoid_as: false

packages/flutter_test/analysis_options.yaml

+include: ../analysis_options.yaml
+
+analyzer:
+  strong-mode:
+    implicit-casts: false
+    implicit-dynamic: false

packages/flutter_tools/analysis_options.yaml

@@ -3,6 +3,11 @@
 
 include: ../../analysis_options.yaml
 
+analyzer:
+  strong-mode:
+    implicit-casts: false
+    implicit-dynamic: false
+
 linter:
   rules:
     unawaited_futures: true

packages/flutter_web_plugins/analysis_options.yaml

+# Override the parent analysis_options until all the repo has implicit-casts: false
+
+include: ../analysis_options.yaml
+
+analyzer:
+  strong-mode:
+    implicit-casts: false
+    implicit-dynamic: false

packages/fuchsia_remote_debug_protocol/analysis_options.yaml

+# Override the parent analysis_options until all the repo has implicit-casts: false
+
+include: ../analysis_options.yaml
+
+analyzer:
+  strong-mode:
+    implicit-casts: false
+    implicit-dynamic: false