Restore r1383: Fix method resolution to warn about things that don't exist on the declared type.

frog/gen.dart

Index: frog/gen.dart
diff --git a/frog/gen.dart b/frog/gen.dart
index bdc16fedc3b0bddf369e75f48819ddfe678b5293..0c2d29707e9eb32f4ce2dd8f0c2807455d3e6418 100644
--- a/frog/gen.dart
+++ b/frog/gen.dart
@@ -360,7 +360,7 @@ function $inherits(child, parent) {
   List _orderValues(Map map) {
     // TODO(jmesserly): should we copy the list?
     // Right now, the Maps are returning a copy already.
-    final values = map.getValues();
+    List values = map.getValues();
     values.sort(_compareMembers);
     return values;
   }
@@ -572,7 +572,8 @@ class MethodGenerator implements TreeVisitor {
     // For named lambdas, add the name to this scope so we can call it
     // recursively.
     if (enclosingMethod != null && method.name != '') {
-       _scope.create(method.name, method.functionType, method.definition);
+      MethodMember m = method; // lambdas must be MethodMembers
+      _scope.create(m.name, m.functionType, m.definition);
     }
     _usedTemps = new Set();
     _freeTemps = [];
@@ -715,16 +716,19 @@ class MethodGenerator implements TreeVisitor {
 
     _provideOptionalParamInfo(defWriter);
 
-    if (method is MethodMember && method._providePropertySyntax) {
-      defWriter.enterBlock(
-        '${method.declaringType.jsname}.prototype.get\$${method.jsname} = function() {');
-      // TODO(jimhug): Bind not availabe in Safari, need fallback.
-      defWriter.writeln(
-        'return ${method.declaringType.jsname}.prototype.${method.jsname}.bind(this);');
-      defWriter.exitBlock('}');
-
-      if (method._provideFieldSyntax) {
-        world.internalError('bound method accessed with field syntax');
+    if (method is MethodMember) {
+      MethodMember m = method;
+      if (m._providePropertySyntax) {
+        defWriter.enterBlock('${m.declaringType.jsname}.prototype'
+            + '.get\$${m.jsname} = function() {');
+        // TODO(jimhug): Bind not available in older Safari, need fallback?
+        defWriter.writeln('return ${m.declaringType.jsname}.prototype.'
+            + '${m.jsname}.bind(this);');
+        defWriter.exitBlock('}');
+
+        if (m._provideFieldSyntax) {
+          world.internalError('bound m accessed with field syntax');
+        }
       }
     }
   }
@@ -735,32 +739,35 @@ class MethodGenerator implements TreeVisitor {
    * generated to support run time stub creation.
    */
   _provideOptionalParamInfo(CodeWriter defWriter) {
-    if (method is MethodMember && method._provideOptionalParamInfo) {
-      var optNames = [];
-      var optValues = [];
-      method.genParameterValues();
-      for (var param in method.parameters) {
-        if (param.isOptional) {
-          optNames.add(param.name);
-          optValues.add(_escapeString(param.value.code));
-        }
-      }
-      if (optNames.length > 0) {
-        // TODO(jmesserly): the logic for how to refer to
-        // static/instance/top-level members is duplicated all over the place.
-        // Badly needs cleanup.
-        var start = '';
-        if (method.isStatic) {
-          if (!method.declaringType.isTop) {
-            start = method.declaringType.jsname + '.';
+    if (method is MethodMember) {
+      MethodMember meth = method;
+      if (meth._provideOptionalParamInfo) {
+        var optNames = [];
+        var optValues = [];
+        meth.genParameterValues();
+        for (var param in meth.parameters) {
+          if (param.isOptional) {
+            optNames.add(param.name);
+            optValues.add(_escapeString(param.value.code));
           }
-        } else {
-          start = method.declaringType.jsname + '.prototype.';
         }
+        if (optNames.length > 0) {
+          // TODO(jmesserly): the logic for how to refer to
+          // static/instance/top-level members is duplicated all over the place.
+          // Badly needs cleanup.
+          var start = '';
+          if (meth.isStatic) {
+            if (!meth.declaringType.isTop) {
+              start = meth.declaringType.jsname + '.';
+            }
+          } else {
+            start = meth.declaringType.jsname + '.prototype.';
+          }
 
-        optNames.addAll(optValues);
-        var optional = "['" + Strings.join(optNames, "', '") + "']";
-        defWriter.writeln('${start}${method.jsname}.\$optional = $optional');
+          optNames.addAll(optValues);
+          var optional = "['" + Strings.join(optNames, "', '") + "']";
+          defWriter.writeln('${start}${meth.jsname}.\$optional = $optional');
+        }
       }
     }
   }
@@ -967,7 +974,8 @@ class MethodGenerator implements TreeVisitor {
   /** Visits [body] without creating a new block for a [BlockStatement]. */
   bool visitStatementsInBlock(Statement body) {
     if (body is BlockStatement) {
-      for (var stmt in body.body) {
+      BlockStatement block = body;
+      for (var stmt in block.body) {
         stmt.visit(this);
       }
     } else {
@@ -1534,26 +1542,27 @@ class MethodGenerator implements TreeVisitor {
     var position = node.target;
     var name = '\$call';
     if (node.target is DotExpression) {
-      target = node.target.self.visit(this);
-      name = node.target.name.name;
-      position = node.target.name;
+      DotExpression dot = node.target;
+      target = dot.self.visit(this);
+      name = dot.name.name;
+      position = dot.name;
     } else if (node.target is VarExpression) {
-      name = node.target.name.name;
+      VarExpression varExpr = node.target;
+      name = varExpr.name.name;
       var meth = method.declaringType.resolveMember(name);
       if (meth != null) {
         target = _makeThisOrType();
-        return meth.invoke(this, node.target, target,
+        return meth.invoke(this, varExpr, target,
             _makeArgs(node.arguments));
       }
       // Look for members of the top-level type (or imported libs).
-      meth = method.declaringType.library.lookup(name, node.target.span);
+      meth = method.declaringType.library.lookup(name, varExpr.span);
       if (meth != null) {
-        return meth.invoke(this, node.target, null,
-            _makeArgs(node.arguments));
+        return meth.invoke(this, varExpr, null, _makeArgs(node.arguments));
       }
 
       name = '\$call';
-      target = node.target.visit(this);
+      target = varExpr.visit(this);
     } else {
       target = node.target.visit(this);
     }