Support fast noSuchMethod dispatch for any number of arguments.

tests/language/no_such_method_dispatcher_test.dart

Index: tests/language/no_such_method_dispatcher_test.dart
===================================================================
--- tests/language/no_such_method_dispatcher_test.dart	(revision 24510)
+++ tests/language/no_such_method_dispatcher_test.dart	(working copy)
@@ -16,6 +16,35 @@
 
 class B extends A { }
 
+
+call_bar(x) => x.bar();
+
+testMessage() {
+  try {
+    call_bar(5);
+  } catch (e) {
+    Expect.isTrue(e.toString().indexOf("method not found") != -1);

[ahe, 2013/07/20 17:24:27]

I'm working on improving dart2js' handling of NoSuchMethod errors.

Based on feedback from Lars, the error message has been changed to for example:

  NoSuchMethodError: Cannot call "bar$0" on "5" (Object 5 has no method 'bar$0')

The latter part is the message from the JavaScript VM, and the first part
represents what we have extracted from that message.

However, this varies a lot depending on how much information the browser
provides.  For example, if we cannot identify the method name, it could just be:

  NoSuchMethodError: Browser-specific error message.

Looking at the VM implementation of NoSuchMethodError.toString, I can see that
this particular message only occur in certain situations, and this might be
important for this test.

Can you elaborate on why this test is expecting this particular message?

+  }
+}
+
+class C {
+  C(this.pos, this.named, this.posArgs, this.namedArgs);
+  var pos, named;
+  noSuchMethod(m) {
+    Expect.equals(pos, m.positionalArguments.length);
+    Expect.equals(named, m.namedArguments.length);
+    for (var i = 0; i < posArgs.length; ++i) {
+      Expect.equals(posArgs[i], m.positionalArguments[i]);
+    }
+    for (var k in namedArgs.keys) {
+      Expect.equals(namedArgs[k], m.namedArguments[new Symbol(k)]);
+    }
+    return 123;
+  }
+  List posArgs;
+  Map namedArgs;
+}
+
 main() {
   var a = new A();
   for (var i = 0; i < 100; ++i) Expect.equals(123, a.foo());
@@ -28,5 +57,21 @@
     Expect.equals(123, b.bar());
     Expect.equals(2, b.bar(1));
   }
+
+  for (var i = 0; i < 100; ++i) {
+    Expect.equals(123, b.bar(1,2,3));
+    Expect.equals(123, b.bar(1,2,foo:3));
+  }
+
+  // Test named and positional arguments.
+  var c = new C(1, 2, [100], {"n1":101, "n2":102});
+  for (var i = 0; i < 100; ++i) {
+    Expect.equals(123, c.bar(100, n1:101, n2:102));
+    Expect.equals(123, c.bar(100, n2:102, n1:101));
+  }
+
+  // Test NoSuchMethodError message.
+  for (var i = 0; i < 100; i++) testMessage();
+
 }