Change resolving of instance methods to check early for name mismatch.

runtime/vm/resolver_test.cc

 // Copyright (c) 2012, 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.
 
 #include "platform/assert.h"
 #include "vm/assembler.h"
 #include "vm/class_finalizer.h"
 #include "vm/compiler.h"
 #include "vm/dart_entry.h"
 #include "vm/object.h"
@@ skipping 152 matching lines @@
   ASSERT(!lib.IsNull());
   const Class& cls = Class::Handle(lib.LookupClass(
       String::Handle(Symbols::New(test_class_name))));
   EXPECT(!cls.IsNull());
 
   Instance& receiver = Instance::Handle(Instance::New(cls));
   const String& function_name = String::Handle(String::New(test_function_name));
 
   // Now try to resolve and invoke the instance function in this class.
   {
-    const int kNumPositionalArguments = 3;
-    const int kNumNamedArguments = 0;
+    const int kNumArguments = 3;
+    ArgumentsDescriptor args_desc(
+        Array::Handle(ArgumentsDescriptor::New(kNumArguments)));
     const Function& function = Function::Handle(
         Resolver::ResolveDynamic(receiver,
                                  function_name,
-                                 kNumPositionalArguments,
-                                 kNumNamedArguments));
+                                 args_desc));
     EXPECT(!function.IsNull());
-    const Array& args = Array::Handle(Array::New(kNumPositionalArguments));
+    const Array& args = Array::Handle(Array::New(kNumArguments));
     args.SetAt(0, receiver);
     const String& arg0 = String::Handle(String::New("junk"));
     args.SetAt(1, arg0);
     const Smi& arg1 = Smi::Handle(Smi::New(kTestValue));
     args.SetAt(2, arg1);
     const Smi& retval = Smi::Handle(
         reinterpret_cast<RawSmi*>(DartEntry::InvokeFunction(function, args)));
     EXPECT_EQ(kTestValue, retval.Value());
   }
 
   // Now try to resolve an instance function with invalid argument count.
   {
-    const int kNumPositionalArguments = 1;
-    const int kNumNamedArguments = 0;
+    const int kNumArguments = 1;
+    ArgumentsDescriptor args_desc(
+        Array::Handle(ArgumentsDescriptor::New(kNumArguments)));
     const Function& bad_function = Function::Handle(
         Resolver::ResolveDynamic(receiver,
                                  function_name,
-                                 kNumPositionalArguments,
-                                 kNumNamedArguments));
+                                 args_desc));
     EXPECT(bad_function.IsNull());
   }
 
   // Hierarchy walking.
   {
-    const int kNumPositionalArguments = 1;
-    const int kNumNamedArguments = 0;
+    const int kNumArguments = 1;
+    ArgumentsDescriptor args_desc(
+        Array::Handle(ArgumentsDescriptor::New(kNumArguments)));
     const String& super_function_name =
         String::Handle(String::New("dynCall"));
     const Function& super_function = Function::Handle(
         Resolver::ResolveDynamic(receiver,
                                  super_function_name,
-                                 kNumPositionalArguments,
-                                 kNumNamedArguments));
+                                 args_desc));
     EXPECT(!super_function.IsNull());
   }
 }
 
 }  // namespace dart