Reduce strong mode errors and warnings

pkg/front_end/lib/src/fasta/analyzer/mock_element.dart

Index: pkg/front_end/lib/src/fasta/analyzer/mock_element.dart
diff --git a/pkg/front_end/lib/src/fasta/analyzer/mock_element.dart b/pkg/front_end/lib/src/fasta/analyzer/mock_element.dart
index a55197bff6615dd26ea82cad57a9d4caf8ced2ce..40154ee803b498b2ad142c7194347cfda61507fb 100644
--- a/pkg/front_end/lib/src/fasta/analyzer/mock_element.dart
+++ b/pkg/front_end/lib/src/fasta/analyzer/mock_element.dart
@@ -4,7 +4,12 @@
 
 library fasta.analyzer.mock_element;
 
+import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/element/element.dart';
+import 'package:analyzer/dart/element/type.dart';
+import 'package:analyzer/dart/constant/value.dart';
+import 'package:analyzer/src/generated/source.dart' show
+     SourceRange;
 
 import '../errors.dart' show
     internalError;
@@ -60,13 +65,14 @@ abstract class MockElement implements Element {
 
   get unit => internalError("not supported.");
 
-  accept(visitor) => internalError("not supported.");
+  T accept<T>(ElementVisitor<T> visitor) => internalError("not supported.");

[ahe, 2017/02/02 07:28:30]

The types added in this file should be inferred from supertypes.

[Siggi Cherem (dart-lang), 2017/02/02 16:30:35]

Agree - I was surprised that this wasn't working. I'll follow up to see why.

[Siggi Cherem (dart-lang), 2017/02/06 23:35:50]

Fixed. There were two issues:
 - some methods were declared in this class but not in `Element`, so there was
nothing to inherit in those cases, I moved those to the subclasses where they
are first needed and added an extra interface impementation to MockElement as
well.
 - arity of type-arguments had to match too. In this case I had to add <T>, but
I can still omit the types on the arguments and return value.

 
   String computeDocumentationComment() => internalError("not supported.");
 
-  computeNode() => internalError("not supported.");
+  AstNode computeNode() => internalError("not supported.");
 
-  Element getAncestor(predicate) => internalError("not supported.");
+  E getAncestor<E extends Element>(bool predicate(Element e))
+    => internalError("not supported.");
 
   String getExtendedDisplayName(String shortName) {
     return internalError("not supported.");
@@ -84,13 +90,14 @@ abstract class MockElement implements Element {
 
   int get uriOffset => internalError("not supported.");
 
-  get type => null;
+  DartType get type => null;
 
-  get typeParameters => internalError("not supported.");
+  List<TypeParameterElement> get typeParameters =>
+    internalError("not supported.");
 
   List<ParameterElement> get parameters => internalError("not supported.");
 
-  get returnType => internalError("not supported.");
+  DartType get returnType => internalError("not supported.");
 
   List<FunctionElement> get functions => internalError("not supported.");
 
@@ -116,9 +123,9 @@ abstract class MockElement implements Element {
     return internalError("not supported.");
   }
 
-  get visibleRange => internalError("not supported.");
+  SourceRange get visibleRange => internalError("not supported.");
 
-  get constantValue => internalError("not supported.");
+  DartObject get constantValue => internalError("not supported.");
 
   bool get hasImplicitType => internalError("not supported.");
 
@@ -132,7 +139,7 @@ abstract class MockElement implements Element {
 
   bool get isPotentiallyMutatedInScope => internalError("not supported.");
 
-  computeConstantValue() => internalError("not supported.");
+  DartObject computeConstantValue() => internalError("not supported.");
 }
 
 abstract class MockLibraryElement extends MockElement
@@ -202,6 +209,8 @@ abstract class MockCompilationUnitElement extends MockElement
     return internalError("not supported.");
   }
 
+  LibraryElement get enclosingElement => internalError("not supported.");
+
   List<ClassElement> get enums => internalError("not supported.");
 
   List<FunctionElement> get functions => internalError("not supported.");
@@ -223,6 +232,8 @@ abstract class MockCompilationUnitElement extends MockElement
   ClassElement getEnum(String name) => internalError("not supported.");
 
   ClassElement getType(String name) => internalError("not supported.");
+
+  CompilationUnit computeNode() => internalError("not supported.");
 }
 
 abstract class MockClassElement extends MockElement implements ClassElement {
@@ -325,6 +336,10 @@ abstract class MockClassElement extends MockElement implements ClassElement {
       String setterName, LibraryElement library) {
     return internalError("not supported.");
   }
+
+  NamedCompilationUnitMember computeNode() => internalError("not supported.");
+
+  InterfaceType get type => internalError("not supported.");
 }
 
 abstract class MockFunctionElement extends MockElement
@@ -333,6 +348,10 @@ abstract class MockFunctionElement extends MockElement
       : super(ElementKind.FUNCTION);
 
   bool get isEntryPoint => internalError("not supported.");
+
+  FunctionType get type => internalError("not supported.");
+
+  FunctionDeclaration computeNode() => internalError("not supported.");
 }
 
 abstract class MockParameterElement extends MockElement
@@ -350,11 +369,9 @@ abstract class MockParameterElement extends MockElement
 
   List<ParameterElement> get parameters => internalError("not supported.");
 
-  List<TypeParameterElement> get typeParameters {
-    return internalError("not supported.");
-  }
-
   void appendToWithoutDelimiters(StringBuffer buffer) {
     return internalError("not supported.");
   }
+
+  FormalParameter computeNode() => internalError("not supported.");
 }