Don't call dinvoke on Object methods

lib/src/checker/rules.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library dev_compiler.src.checker.rules;
 
 import 'package:analyzer/src/generated/ast.dart';
 import 'package:analyzer/src/generated/element.dart';
 import 'package:analyzer/src/generated/resolver.dart';
 
@@ skipping 26 matching lines @@
   bool maybeNonNullableType(DartType t) => false;
 
   StaticInfo checkAssignment(Expression expr, DartType t, bool constContext);
 
   DartType getStaticType(Expression expr) => expr.staticType;
 
   DartType elementType(Element e);
 
   bool isDynamic(DartType t);
   bool isDynamicTarget(Expression expr);
-  bool isDynamicGet(Expression expr);
+  bool isDynamicGet(Expression target, SimpleIdentifier name);
   bool isDynamicCall(Expression call);
 }
 
 class DartRules extends TypeRules {
   DartRules(TypeProvider provider) : super(provider);
 
   bool isSubTypeOf(DartType t1, DartType t2) {
     return t1.isSubtypeOf(t2);
   }
 
@@ skipping 10 matching lines @@
     return null;
   }
 
   DartType elementType(Element e) {
     return (e as dynamic).type;
   }
 
   /// By default, all invocations are dynamic in Dart.
   bool isDynamic(DartType t) => true;
   bool isDynamicTarget(Expression expr) => true;
-  bool isDynamicGet(Expression expr) => true;
+  bool isDynamicGet(Expression target, SimpleIdentifier name) => true;
   bool isDynamicCall(Expression call) => true;
 }
 
 class RestrictedRules extends TypeRules {
   final CheckerReporter _reporter;
   final RulesOptions options;
   final List<DartType> _nonnullableTypes;
   DownwardsInference inferrer;
 
   DartType _typeFromName(String name) {
@@ skipping 434 matching lines @@
   }
 
   bool isDynamic(DartType t) => options.ignoreTypes || t.isDynamic;
 
   /// Returns `true` if the target expression is dynamic.
   bool isDynamicTarget(Expression expr) =>
       options.ignoreTypes || getStaticType(expr).isDynamic;
 
   /// Returns `true` if the expression is a dynamic property access or prefixed
   /// identifier.
-  bool isDynamicGet(Expression expr) {
+  bool isDynamicGet(Expression target, SimpleIdentifier name) {
     if (options.ignoreTypes) return true;
-    var t = getStaticType(expr);
+    if (!name.staticType.isDynamic) return false;
+    var t = getStaticType(target);
     // TODO(jmesserly): we should not allow all property gets on `Function`
     return t.isDynamic || t.isDartCoreFunction;
   }
 
   /// Returns `true` if the expression is a dynamic function call or method
   /// invocation.
   bool isDynamicCall(Expression call) {
     if (options.ignoreTypes) return true;
     var t = getStaticType(call);
     // TODO(jmesserly): fix handling of types with `call` methods. These are not
@@ skipping 282 matching lines @@
     var entries = e.entries;
     bool inferEntry(MapLiteralEntry entry) {
       return _inferExpression(entry.key, kType, errors) &&
           _inferExpression(entry.value, vType, errors);
     }
     var b = entries.every(inferEntry);
     if (b) annotateMapLiteral(e, targs);
     return b;
   }
 }